Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/24504 )
Change subject: IMPALA-15101: Add UUID primitive type to Impala ...................................................................... Patch Set 2: (1 comment) Thanks for working on this! http://gerrit.cloudera.org:8080/#/c/24504/2/be/src/runtime/types.h File be/src/runtime/types.h: http://gerrit.cloudera.org:8080/#/c/24504/2/be/src/runtime/types.h@102 PS2, Line 102: is_uuid_(false) Could you please explore the case when we add a standalone enum (TYPE_UUID), and in the backend we store the fixed 16 bytes inline in the tuple (like CHAR(N) or large DECIMALs)? It might require a bigger code change. but I think the changes will be mostly trivial/mechanical, with AI it shouldn't be too hard. It would make read/write operations quite straightforward as well. -- To view, visit http://gerrit.cloudera.org:8080/24504 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iefc73aefe73b6144c929ec37b0cb333007cf8bfe Gerrit-Change-Number: 24504 Gerrit-PatchSet: 2 Gerrit-Owner: Arnab Karmakar <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Thu, 25 Jun 2026 17:09:47 +0000 Gerrit-HasComments: Yes
