-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
planner: Fixup error when assign DEFAULT
in INSERT/UPDATE/REPLACE ... SET ...
statement
#13211
Conversation
Codecov Report
@@ Coverage Diff @@
## master #13211 +/- ##
=========================================
Coverage 80.156% 80.156%
=========================================
Files 472 472
Lines 116786 116786
=========================================
Hits 93611 93611
Misses 15915 15915
Partials 7260 7260 |
/run-all-tests |
DEFAULT
in UPDATE
statementDEFAULT
in INSERT/UPDATE/REPLACE ... SET ...
statement
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.
The test case is clear, nice. Rest LGTM
/run-all-tests |
Did you test |
@djshow832 Yes, In #13168, PTAL |
Co-Authored-By: djshow832 <873581766@qq.com>
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.
LGTM
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.
LGTM
@AilinKid, @djshow832, @wjhuang2016, @lamxTyler, @qw4990, @XuHuaiyu, PTAL. |
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.
LGTM
/run-all-tests |
/run-all-tests |
cherry pick to release-3.0 failed |
cherry pick to release-2.1 failed |
cherry pick to release-3.1 failed |
What problem does this PR solve?
Fix #11939
What is changed and how it works?
In statement
update t1 set a = default
, When we buildUpdate
logic plan, we do:defaultExpr.Name
byassign.Column
before rewrite its assignment expression.DEAFULT
, just ignore this column.The value specified for generated column 'gc' in table 't' is not allowed.
Check List
Tests
Code changes
Side effects
Related changes
Release note
DEAFULT
inINSERT/UPDATE/REPLACE ... SET xxx=DEFAULT
statement