We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
MIN()/MAX()
See reproducer in datafusion-cli
DataFusion CLI v41.0.0 > select min(1,2); thread 'main' panicked at /Users/yongting/Desktop/code/my_datafusion/arrow-datafusion/datafusion/functions-aggregate/src/min_max.rs:71:5: assertion `left == right` failed left: 2 right: 1
DataFusion CLI v41.0.0 > select max(2,1); thread 'main' panicked at /Users/yongting/Desktop/code/my_datafusion/arrow-datafusion/datafusion/functions-aggregate/src/min_max.rs:71:5: assertion `left == right` failed left: 2 right: 1 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
No response
Note those queries are invalid since MIN/MAX only takes one argument, but it should still return an error instead of crash
MIN/MAX
Found by SQLancer #11030
The text was updated successfully, but these errors were encountered:
Both postgres and duckdb does not support min/max more than one argument. I suggest we leave it alone too.
Sorry, something went wrong.
I have the same thought, but I think it would be nice if we throw an error with a message instead of panic and crash.
I have updated the issue for clarification (expecting an error instead of crash) BTW I think this is introduced by a recent change
min
max
Successfully merging a pull request may close this issue.
Describe the bug
See reproducer in datafusion-cli
To Reproduce
No response
Expected behavior
Note those queries are invalid since
MIN/MAX
only takes one argument, but it should still return an error instead of crashAdditional context
Found by SQLancer #11030
The text was updated successfully, but these errors were encountered: