-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add presto function 'to_base32' #8652
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for meta-velox canceled.
|
c6f4282
to
d976584
Compare
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.
@Joe-Abraham Thanks for splitting PRs. Please also add Presto e2e tests for these functions here: https://github.com/prestodb/presto/blob/master/presto-native-execution/src/test/java/com/facebook/presto/nativeworker/AbstractTestNativeGeneralQueries.java
@karteekmurthys e2e test cases were added prestodb/presto#21537. Waiting for the velox changes to be merged and pulled to presto. |
1be1df0
to
d5c5439
Compare
f39d90e
to
57c0d45
Compare
e3f2488
to
6d23b3b
Compare
d6fc45e
to
dec1883
Compare
0349258
to
32d15d5
Compare
32d15d5
to
c6461d7
Compare
c6461d7
to
80e53ed
Compare
Add presto function 'to_base32'