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.

Sign up for GitHub

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

Jump to bottom

[CALCITE-5286] Join with parameterized LIMIT throws AssertionError "n… #2921

Merged
merged 1 commit into from Oct 3, 2022
Merged

[CALCITE-5286] Join with parameterized LIMIT throws AssertionError "n… #2921

merged 1 commit into from Oct 3, 2022

Conversation

JiajunBernoulli
Copy link
Contributor

…ot a literal"

@JiajunBernoulli JiajunBernoulli force-pushed the calcite-5286-join branch 2 times, most recently from 36dd45e to 26b64a5 Compare September 29, 2022 12:31
@rubenada
Copy link
Contributor

LGTM

@rubenada rubenada added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label Sep 30, 2022
@rubenada rubenada merged commit 3a38ebf into apache:main Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
LGTM-will-merge-soon Overall PR looks OK. Only minor things left.
Projects
None yet
Milestone
No milestone
2 participants
@JiajunBernoulli @rubenada