Skip to content

[CALCITE-2954] SubQueryJoinRemoveRule and SubQueryProjectRemoveRule pa…#1135

Closed
vineetgarg02 wants to merge 1 commit intoapache:masterfrom
vineetgarg02:CALCITE-2954
Closed

[CALCITE-2954] SubQueryJoinRemoveRule and SubQueryProjectRemoveRule pa…#1135
vineetgarg02 wants to merge 1 commit intoapache:masterfrom
vineetgarg02:CALCITE-2954

Conversation

@vineetgarg02
Copy link
Contributor

No description provided.

…ssing on empty set instead of set of correlation id (Vineet Garg)
Copy link
Member

@hsyuan hsyuan left a comment

Choose a reason for hiding this comment

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

LGTM

@hsyuan hsyuan added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label Apr 2, 2019
@hsyuan hsyuan changed the title CALCITE-2954: SubQueryJoinRemoveRule and SubQueryProjectRemoveRule pa… [CALCITE-2954] SubQueryJoinRemoveRule and SubQueryProjectRemoveRule pa… Apr 2, 2019
@hsyuan hsyuan closed this in 6d57ff9 Apr 5, 2019
vvysotskyi pushed a commit to mapr/incubator-calcite that referenced this pull request Apr 19, 2019
…assing on empty set instead of set of correlation id (Vineet Garg)

Close apache#1135

(cherry picked from commit 6d57ff9)
vvysotskyi pushed a commit to mapr/incubator-calcite that referenced this pull request Apr 19, 2019
…assing on empty set instead of set of correlation id (Vineet Garg)

Close apache#1135

(cherry picked from commit 6d57ff9)
vvysotskyi pushed a commit to mapr/incubator-calcite that referenced this pull request Apr 19, 2019
…assing on empty set instead of set of correlation id (Vineet Garg)

Close apache#1135

(cherry picked from commit 6d57ff9)
vvysotskyi pushed a commit to mapr/incubator-calcite that referenced this pull request Apr 19, 2019
…assing on empty set instead of set of correlation id (Vineet Garg)

Close apache#1135

(cherry picked from commit 6d57ff9)
KazydubB pushed a commit to KazydubB/calcite that referenced this pull request Apr 22, 2019
…assing on empty set instead of set of correlation id (Vineet Garg)

Close apache#1135
wangxlong pushed a commit to wangxlong/calcite that referenced this pull request Feb 13, 2020
…assing on empty set instead of set of correlation id (Vineet Garg)

Close apache#1135
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Jun 18, 2021
…assing on empty set instead of set of correlation id (Vineet Garg)

Close apache#1135

Change-Id: If5c850e111271c03179ee0167d752b5a041f5fce
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Mar 16, 2026
…assing on empty set instead of set of correlation id (Vineet Garg)

Close apache#1135

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

Labels

LGTM-will-merge-soon Overall PR looks OK. Only minor things left.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants