-
Notifications
You must be signed in to change notification settings - Fork 288
fix(plan): handle tuple IN/NOT IN #23591
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
|
aunjgr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
需要加一些带null值的测试。根据过去经验,对null值的处理最容易出错。
What type of PR is this?
Which issue(s) this PR fixes:
issue #14490
What this PR does / why we need it:
Fixes tuple IN/NOT IN handling in the binder so row constructors like (a,b) IN ((1,2),(3,4)) no longer fall back to scalar IN and error. Adds SQL and Go tests to prevent regressions.