Add UUID to Varbinary casting

facebookincubator / velox

A composable and fully extensible C++ execution engine library for data management systems.

https://velox-lib.io/

Apache License 2.0

3.53k stars 1.16k forks source link

Add UUID to Varbinary casting #10585

Open aditi-pandit opened 3 months ago

aditi-pandit commented 3 months ago

Description

Presto UUID is typically stored with VARCHAR or VARBINARY with Hive metastore and is accessed so in Dwrf/Parquet readers. VARBINARY is a more compact representation than VARCHAR. But UUID can't be cast from and to it today.

This is supported in Presto Java https://github.com/prestodb/presto/blob/master/presto-main/src/main/java/com/facebook/presto/type/UuidOperators.java#L155

SQL failure https://github.com/prestodb/presto/pull/23301/files#r1693439871

SteNicholas commented 3 months ago

@aditi-pandit, does anyone take this first good issue? If no, I will try to take this ticket to start contribution.

aditi-pandit commented 3 months ago

HI @SteNicholas, Sure. Feel free to take this ticket. Assigned it to you.