-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[CALCITE-5836] Implement Rel2Sql for MERGE #3397
Conversation
core/src/test/java/org/apache/calcite/rel/rel2sql/RelToSqlConverterTest.java
Show resolved
Hide resolved
I made a rebase to solve problem of hundred of commits in this PR. |
Hi @JiajunBernoulli, Do you want to continue with the code review? If you think this PR is ok, do you consider merging it? |
final SqlNode source = join.getLeft(); | ||
|
||
SqlUpdate update = null; | ||
List<String> updateColumnList = |
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.
Some variables can be final
, please use final
as much as possible.
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.
I have fixed this and squashed the commits.
Co-authored-by: Greg Hart <[email protected]>
Kudos, SonarCloud Quality Gate passed! |
Close #3310 |
Support MERGE INTO cluase for JDBC adapter