Skip to content
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

pushdown power to TiFlash/TiKV #50485

Closed
windtalker opened this issue Jan 17, 2024 · 0 comments · Fixed by #50486
Closed

pushdown power to TiFlash/TiKV #50485

windtalker opened this issue Jan 17, 2024 · 0 comments · Fixed by #50486
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@windtalker
Copy link
Contributor

windtalker commented Jan 17, 2024

Enhancement

TiDB already pushdown pow to TiFlash

ast.Sqrt, ast.Log, ast.Log2, ast.Log10, ast.Ln, ast.Exp, ast.Pow, ast.Sign,

And TiKV
ast.Ceil, ast.Ceiling, ast.Floor, ast.Sqrt, ast.Sign, ast.Ln, ast.Log, ast.Log2, ast.Log10, ast.Exp, ast.Pow,

According to MySQL's doc, power is just a synonym for pow, so it is ok to pushdown power to TiFlash/TiKV without extra work in TiFlash/TiKV side.

@windtalker windtalker added the type/enhancement The issue or PR belongs to an enhancement. label Jan 17, 2024
@windtalker windtalker changed the title pushdown power to TiFlash pushdown power to TiFlash/TiKV Jan 17, 2024
ti-chi-bot bot pushed a commit that referenced this issue Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant