Skip to content

Conversation

@huaxingao
Copy link
Contributor

What changes were proposed in this pull request?

back porting PR to 3.3.
Only set KeyGroupedPartitioning when the referenced column is in the output

Why are the changes needed?

bug fixing

Does this PR introduce any user-facing change?

No

How was this patch tested?

New test

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM (Pending CIs)

@dongjoon-hyun
Copy link
Member

Thank you, @huaxingao .

@github-actions github-actions bot added the SQL label Sep 15, 2022
@dongjoon-hyun
Copy link
Member

Merged to branch-3.3.

dongjoon-hyun pushed a commit that referenced this pull request Sep 15, 2022
…renced column is in the output

### What changes were proposed in this pull request?
back porting [PR](#37886) to 3.3.
Only set `KeyGroupedPartitioning` when the referenced column is in the output

### Why are the changes needed?
bug fixing

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
New test

Closes #37901 from huaxingao/3.3.

Authored-by: huaxingao <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
@huaxingao
Copy link
Contributor Author

Thanks a lot @dongjoon-hyun

@huaxingao huaxingao deleted the 3.3 branch September 15, 2022 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants