Skip to content
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

Ensure Parameters are in proper order for queries having WITH clause #17083

Merged
merged 1 commit into from
Mar 1, 2022

Conversation

v-jizhang
Copy link
Contributor

Cherry-pick of trinodb/trino#1529

Fixes #17012

Co-authored-by: praveenkrishna praveenkrishna@tutanota.com

Test plan - Added a test.

== RELEASE NOTES ==

General Changes
* Ensure Parameters are in proper order for queries having WITH clause

@rubenssoto
Copy link

Is there any chance of being merged in the next version? Very important fix for us.

Thank you!

@zhenxiao
Copy link
Collaborator

hi @v-jizhang could you please rebase this PR? I will take a look

@v-jizhang v-jizhang force-pushed the parameters-proper-order branch from 9c366a8 to a45b56b Compare February 24, 2022 16:30
Cherry-pick of trinodb/trino#1529

Fixes prestodb#17012

Co-authored-by: praveenkrishna <praveenkrishna@tutanota.com>
@v-jizhang v-jizhang force-pushed the parameters-proper-order branch from a45b56b to e7974ed Compare February 24, 2022 17:56
@v-jizhang
Copy link
Contributor Author

hi @v-jizhang could you please rebase this PR? I will take a look
@zhenxiao
Done. Thanks for reviewing.

Copy link
Collaborator

@zhenxiao zhenxiao left a comment

Choose a reason for hiding this comment

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

looks good

@zhenxiao zhenxiao merged commit 25b2f25 into prestodb:master Mar 1, 2022
@varungajjala varungajjala mentioned this pull request Mar 22, 2022
9 tasks
@asjadsyed asjadsyed mentioned this pull request Mar 23, 2022
9 tasks
@asjadsyed asjadsyed mentioned this pull request Apr 1, 2022
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ensure Parameters are in proper order for queries having WITH clause
4 participants