-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Precedence bug with date comparison to date plus interval #3408
Comments
Seems this is an issue in sqlparser-rs, namely in the values in for |
If no one has picked this up yet, I would be happy to work on this. |
I don't believe so, so go ahead. As mentioned in the comment, I believe it is a bug in |
I'm not really able to reproduce the error on the DataFusion side. For example:
where the contents of
Produces the expected output without any errors. I'm able to get the correct results on the Dask-SQL side, too:
Is there maybe another example I should try? |
I get an error with:
|
Thanks @sarahyurick. Perhaps the original issue already got resolved in sqlparser. It makes sense to me to open a new issue for the issue you found. |
This is fixed -- I have a PR open to add some regression test coverage: #4569 |
Describe the bug
d3.d_date > d1.d_date + INTERVAL '5 days'
fails withInt64 AND Boolean' can't be evaluated because there isn't a common type to coerce the types to
d3.d_date > (d1.d_date + INTERVAL '5 days')
worksTo Reproduce
See above
Expected behavior
None
Additional context
None
The text was updated successfully, but these errors were encountered: