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: migrate password function from TiDB #7777
Conversation
98e3365
to
9838b0e
Compare
Signed-off-by: psinghal20 <psinghal20@gmail.com>
Signed-off-by: psinghal20 <psinghal20@gmail.com>
LGTM |
@Fullstop000 PTAL |
let op = Expression::build(&mut ctx, op).unwrap(); | ||
let got = op.eval(&mut ctx, &[]).unwrap(); | ||
let exp = Datum::Bytes(exp_str.as_bytes().to_vec()); | ||
assert_eq!(got, exp, "password('{:?}')", input_str); |
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.
It could be better to add testing for empty input and the warning I think :)
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.
@Fullstop000 I have added a Null case test, similar to the above tests. PTAL.
Signed-off-by: psinghal20 <psinghal20@gmail.com>
@sre-bot /approve |
/merge |
/approve |
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
/merge |
/run-all-tests |
Signed-off-by: psinghal20 psinghal20@gmail.com
What problem does this PR solve?
Issue Number: closes #6776
Problem Summary:
What is changed and how it works?
Migrates the scalar function password from TiDB.
I am not sure of the correct way to report deprecation warning, let me know if any changes are required. @andylokandy PTAL
Check List
Tests