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
Enable 15 push down functions #13496
Conversation
Signed-off-by: Breezewish <me@breeswish.org>
Signed-off-by: Breezewish <me@breeswish.org>
Codecov Report
@@ Coverage Diff @@
## master #13496 +/- ##
=========================================
Coverage 80.089% 80.089%
=========================================
Files 472 472
Lines 115499 115499
=========================================
Hits 92502 92502
Misses 15743 15743
Partials 7254 7254 |
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
/run-all-tests |
Signed-off-by: Breezewish <me@breeswish.org>
/run-all-tests |
Signed-off-by: Breezewish <me@breeswish.org>
/run-all-tests |
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
//case tipb.ScalarFuncSig_RowSig: | ||
case tipb.ScalarFuncSig_SetVar: | ||
f = &builtinSetVarSig{base} | ||
//case tipb.ScalarFuncSig_ValuesDecimal: |
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 comment case means?
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.
Means they are either not implemented by TiDB (so that there is no builtinXxxSig
), or they are implemented but require additional parameters (like {base, xx, yy}
) that I don't know how to fill for now.
/merge |
/run-all-tests |
Signed-off-by: Breezewish me@breeswish.org
What problem does this PR solve?
Enable 15 push down functions
What is changed and how it works?
Related tests:
tikv/copr-test#21
tikv/copr-test#24
tikv/copr-test#25
tikv/copr-test#11
tikv/copr-test#9
tikv/copr-test#32
tikv/copr-test#29
tikv/copr-test#23
Check List
Tests