Arnab Karmakar has posted comments on this change. ( http://gerrit.cloudera.org:8080/24871 )
Change subject: IMPALA-15203: Add aggregate function support for UUID columns ...................................................................... Patch Set 5: (5 comments) http://gerrit.cloudera.org:8080/#/c/24871/4//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/24871/4//COMMIT_MSG@29 PS4, Line 29: Exclude UUID from codegen > Could also mention that hash-table codegen is now turned on for UUID Done http://gerrit.cloudera.org:8080/#/c/24871/4/be/src/exec/aggregator.cc File be/src/exec/aggregator.cc: http://gerrit.cloudera.org:8080/#/c/24871/4/be/src/exec/aggregator.cc@450 PS4, Line 450: FIXED_UDA_INTERMEDIATE and > nit: could also mention UUID Done http://gerrit.cloudera.org:8080/#/c/24871/4/fe/src/main/java/org/apache/impala/analysis/ComputeStatsStmt.java File fe/src/main/java/org/apache/impala/analysis/ComputeStatsStmt.java: http://gerrit.cloudera.org:8080/#/c/24871/4/fe/src/main/java/org/apache/impala/analysis/ComputeStatsStmt.java@969 PS4, Line 969: return !t.isValid() || !t.isSupported() || t.isComplexOrVariantType() : || t.isUuid() || c.isHidden(); : } > Or we could add UUID here. Done http://gerrit.cloudera.org:8080/#/c/24871/4/fe/src/main/java/org/apache/impala/catalog/BuiltinsDb.java File fe/src/main/java/org/apache/impala/catalog/BuiltinsDb.java: http://gerrit.cloudera.org:8080/#/c/24871/4/fe/src/main/java/org/apache/impala/catalog/BuiltinsDb.java@552 PS4, Line 552: AggIfFinalizeIN10 > This doesn't exist in the BE, we can remove this entry. Done http://gerrit.cloudera.org:8080/#/c/24871/4/tests/query_test/test_iceberg.py File tests/query_test/test_iceberg.py: http://gerrit.cloudera.org:8080/#/c/24871/4/tests/query_test/test_iceberg.py@103 PS4, Line 103: tbl_name = unique_database + ".expire_snapshots" : iceberg_catalogs = IcebergC > Please run this with disable_codegen in [True, False]. Done. Created a new class with the exec option disable_codegen_options=[False, True] -- To view, visit http://gerrit.cloudera.org:8080/24871 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2187495c8be5b95ecff3f458b6549916466e9c83 Gerrit-Change-Number: 24871 Gerrit-PatchSet: 5 Gerrit-Owner: Arnab Karmakar <[email protected]> Gerrit-Reviewer: Arnab Karmakar <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Fri, 25 Sep 2026 05:29:22 +0000 Gerrit-HasComments: Yes
