[CALCITE-5713] Update Operator's operands to ensure consistency with the latest operandList references#3213
[CALCITE-5713] Update Operator's operands to ensure consistency with the latest operandList references#3213gleonSun wants to merge 1 commit intoapache:mainfrom
Conversation
|
Kudos, SonarCloud Quality Gate passed! |
julianhyde
force-pushed
the
main
branch
2 times, most recently
from
8a5cf83 to
cf7f71b
Compare
JiajunBernoulli
added
the
discussion-in-jira
…the latest operandList references
gleonSun
changed the title
|
Kudos, SonarCloud Quality Gate passed! |
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@calcite.apache.org list. Thank you for your contributions. |
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@calcite.apache.org list. Thank you for your contributions. |
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@calcite.apache.org list. Thank you for your contributions. |
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@calcite.apache.org list. Thank you for your contributions. |
2 participants
The SqlBasicCall#set method uses a deep copy in order to modify immutable collections and returns a new operandList object when modified.
If the SQL contains an operation that requires a rewrite call, the deep copy logic will cause the operator and operandList to be different and eventually raise an exception.