Skip to content

[CALCITE-3399] Field-pruning for set operators (except UNION ALL) changes query semantics (Jin Xing)#2125

Closed
Aaaaaaron wants to merge 2 commits intoapache:masterfrom
Aaaaaaron:CALCITE-3399
Closed

[CALCITE-3399] Field-pruning for set operators (except UNION ALL) changes query semantics (Jin Xing)#2125
Aaaaaaron wants to merge 2 commits intoapache:masterfrom
Aaaaaaron:CALCITE-3399

Conversation

@Aaaaaaron
Copy link
Copy Markdown
Member

@Aaaaaaron Aaaaaaron commented Sep 1, 2020

Pre discussion in #1495. The author is no response anymore, so I will promote this.

@Aaaaaaron Aaaaaaron changed the title [CALCITE-3399] Field-pruning for set operators (except UNION ALL) changes query semantics [CALCITE-3399] Field-pruning for set operators (except UNION ALL) changes query semantics (Jin Xing) Sep 1, 2020
@Aaaaaaron Aaaaaaron force-pushed the CALCITE-3399 branch 2 times, most recently from d6bcd23 to 9d7b371 Compare September 1, 2020 05:06
@Aaaaaaron Aaaaaaron requested a review from danny0405 September 2, 2020 07:26
Copy link
Copy Markdown
Contributor

@danny0405 danny0405 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, please resolve the conflicts.

@Aaaaaaron
Copy link
Copy Markdown
Member Author

Aaaaaaron commented Sep 3, 2020

+1, please resolve the conflicts.

Revised and description updated, thanks for your review~

@danny0405 danny0405 added the accepted-and-would-rework-soon This patch is overall ok, but need some refactoring by the committers. label Sep 3, 2020
@danny0405 danny0405 closed this in e0480a9 Sep 4, 2020
MalteBellmann pushed a commit to caespes/calcite that referenced this pull request Feb 21, 2021
…nges query semantics (Jin Xing)

Any SET operations that have comparison between branches can not trim fields.

close apache#2125
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Aug 28, 2025
…nges query semantics (Jin Xing)

Any SET operations that have comparison between branches can not trim fields.

close apache#2125

Change-Id: I2371c17ff9cbcc6f3c4e221bb605ec4c64982b39
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Mar 16, 2026
…nges query semantics (Jin Xing)

Any SET operations that have comparison between branches can not trim fields.

close apache#2125

Change-Id: I2371c17ff9cbcc6f3c4e221bb605ec4c64982b39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted-and-would-rework-soon This patch is overall ok, but need some refactoring by the committers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants