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
Coprocessor: Support SUM() #4797
Coprocessor: Support SUM() #4797
Conversation
Signed-off-by: Breezewish <breezewish@pingcap.com>
…/2_sum Signed-off-by: Breezewish <breezewish@pingcap.com>
/run-integration-tests |
…/2_sum Signed-off-by: Breezewish <breezewish@pingcap.com>
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.
But I realize that the test of both SUM and AVG uses Real type, so the decimal implementation of Summable
is not covered in unit tests. Also, unit tests for cast functions are also missing.
@sticnarf The integration test of AVG is |
/run-integration-tests |
Got it... |
/run-integration-tests |
Signed-off-by: Breezewish <breezewish@pingcap.com>
Signed-off-by: Breezewish <breezewish@pingcap.com>
Signed-off-by: Breezewish <breezewish@pingcap.com>
Signed-off-by: Breezewish <breezewish@pingcap.com>
What have you changed? (mandatory)
This PR adds batch aggregate function SUM() for all data types.
What are the type of the changes? (mandatory)
How has this PR been tested? (mandatory)
New unit tests