-
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 tests constructing maps with duplicate custom TimestampWithTimezones #11228
Add tests constructing maps with duplicate custom TimestampWithTimezones #11228
Conversation
…facebookincubator#11222) Summary: With facebookincubator#11136 array_union and array_remove just work with TimestampWithTimezone. Adding unit tests to verify this and ensure it doesn't break in the future. Differential Revision: D64196032
…ebookincubator#11226) Summary: Update Presto's array_position UDF to work with types that provide custom comparison (both the 2 and 3 argument flavors). We can reuse the implementations for complex types, since that just uses the compare function provided by the Vector. With facebookincubator#11022 this just invokes the Type's custom implementation. Differential Revision: D64209619
…ncubator#11227) Summary: Update Presto's contains UDF to work with types that provide custom comparison. We can reuse the implementation for complex types, since that just uses the compare function provided by the Vector. With facebookincubator#11022 this just invokes the Type's custom implementation. Differential Revision: D64211715
Summary: When constructing a map in Presto and checking for duplicate keys, it already uses the key Vector's equalValuesAt function which calls the custom comparison operator provided by the custom type (if that applies). So this just works automatically for types with custom comparison. Adding unit tests to verify this and ensure it doesn't break in the future. Differential Revision: D64214359
This pull request was exported from Phabricator. Differential Revision: D64214359 |
✅ Deploy Preview for meta-velox canceled.
|
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.
@kevinwilfong thanks!
This pull request has been merged in 5bedca0. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
Summary:
When constructing a map in Presto and checking for duplicate keys, it already uses
the key Vector's equalValuesAt function which calls the custom comparison operator
provided by the custom type (if that applies). So this just works automatically for
types with custom comparison.
Adding unit tests to verify this and ensure it doesn't break in the future.
Differential Revision: D64214359