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
[function] compatible with mysql when aggregate function work on empty data #848
Conversation
Thanks for the contribution! Please review the labels and make any necessary changes. |
Hello @zhaox1n, 🎉 Thank you for opening the pull request! 🎉 |
2 similar comments
Hello @zhaox1n, 🎉 Thank you for opening the pull request! 🎉 |
Hello @zhaox1n, 🎉 Thank you for opening the pull request! 🎉 |
Hello @zhaox1n, 🎉 Thank you for opening the pull request! 🎉 |
display: "uniq", | ||
func_name: "uniq", | ||
columns: vec![columns[0].clone()], | ||
expect: DataValue::UInt64(Some(0)), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe uniq function should return empty rows?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The uniq function seems to behave the same as count distinct, so I return 0
tests/suites/0_stateless/03_0002_select_aggregator_with_filter.sql
Outdated
Show resolved
Hide resolved
tests/suites/0_stateless/02_00007_function_aggreator_on_empty.result
Outdated
Show resolved
Hide resolved
@ zhaox1n thanks for the contribution. Some advice about this pr:
such as
|
2aa7545
to
7feb64c
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
CI Passed |
6 similar comments
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
13 similar comments
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
CI Passed |
github api rejected the bot again |
I hereby agree to the terms of the CLA available at: https://datafuse.rs/policies/cla/
Summary
Summary about this PR
Changelog
Related Issues
Fixes #771
Test Plan
Unit Tests
Stateless Tests