-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
expression: Support Sqrt, Ceil, Floor and CastIntAsReal push down to TiFlash #25085
expression: Support Sqrt, Ceil, Floor and CastIntAsReal push down to TiFlash #25085
Conversation
No release note, Please follow https://github.com/pingcap/community/blob/master/contributors/release-note-checker.md |
…il-floor-pushdown
No release note, Please follow https://github.com/pingcap/community/blob/master/contributors/release-note-checker.md |
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
/cc @qw4990 |
No release note, Please follow https://github.com/pingcap/community/blob/master/contributors/release-note-checker.md |
/run-check_release_note |
No release note, Please follow https://github.com/pingcap/community/blob/master/contributors/release-note-checker.md |
/run-check_dev_2 |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: 2cfa992
|
/run-all-tests |
/run-check_dev_2 |
/merge |
/run-all-tests |
/run-all-tests |
1 similar comment
/run-all-tests |
/merge |
1 similar comment
/merge |
chabuduodel |
/merge |
/chabuduodel |
cherry pick to release-5.0 in PR #25178 |
so effectively |
What problem does this PR solve?
Related to pingcap/tiflash#1901
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
What's Changed:
Add Floor, Ceil, Sqrt and CastIntAsReal(for pushing down a sqrt for an int) into scalarExprSupportedByFlash list
How it Works:
Related changes
pingcap/docs
/pingcap/docs-cn
:Check List
Tests
Use tiflash/integrated/ops or algernon to run a tidb-server and use explain query to make sure corresponding expression is pushed down
Side effects
Release note