velox icon indicating copy to clipboard operation
velox copied to clipboard

Add support for UUID comparison functions

Open BryanCutler opened this issue 6 months ago • 22 comments

This adds binary comparison functions <, >, <=, >= to the UUID custom data type. Equality functions are already present. Added unit tests for testing a query with comparisons between UUID values.

Also fixes the binary representation of UuidType to be a LE int128_t and adjusted Presto serde to put UUID values in the format that Presto Java expects, as 2 long values with msb first.

From https://github.com/facebookincubator/velox/issues/10584, https://github.com/prestodb/presto/issues/23311

BryanCutler avatar Aug 20 '24 23:08 BryanCutler