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

Extend infer_placeholder_types to support BETWEEN predicates #7703

Merged
merged 1 commit into from
Oct 1, 2023

Conversation

andrelmartins
Copy link
Contributor

Which issue does this PR close?

Closes #7702.

Rationale for this change

What changes are included in this PR?

Add extra inference code for BETWEEN predicate.

Are these changes tested?

Unit test in code.

Are there any user-facing changes?

Parameter type inference now works with statements like SELECT ... WHERE col BETWEEN $1 AND $2.

@github-actions github-actions bot added the sql SQL Planner label Sep 29, 2023
@alamb
Copy link
Contributor

alamb commented Sep 29, 2023

Thank you @andrelmartins -- this looks like a nice improvement to me ❤️ -- I kicked off the CI jobs

@alamb
Copy link
Contributor

alamb commented Oct 1, 2023

CI was failing due to #7705. I think if you merge the branch up from main the CI will pass.

Thanks again @andrelmartins

Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Thank you again @andrelmartins -- we have resolved the CI failure on main, so I think this PR is good to go.

@alamb alamb merged commit f959127 into apache:main Oct 1, 2023
21 of 22 checks passed
Ted-Jiang pushed a commit to Ted-Jiang/arrow-datafusion that referenced this pull request Oct 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sql SQL Planner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support parameter ($1) type inference for BETWEEN predicates
2 participants