Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/24536 )
Change subject: IMPALA-15162: Add GEOMETRY column support for Iceberg and Parquet ...................................................................... Patch Set 12: (2 comments) http://gerrit.cloudera.org:8080/#/c/24536/5/be/src/exec/parquet/hdfs-parquet-table-writer.cc File be/src/exec/parquet/hdfs-parquet-table-writer.cc: http://gerrit.cloudera.org:8080/#/c/24536/5/be/src/exec/parquet/hdfs-parquet-table-writer.cc@224 PS5, Line 224: !type().IsGeometryType(); > A comment like the one above (Min/max statistics are meaningless for geomet added a comment http://gerrit.cloudera.org:8080/#/c/24536/5/be/src/exec/parquet/parquet-metadata-utils.cc File be/src/exec/parquet/parquet-metadata-utils.cc: http://gerrit.cloudera.org:8080/#/c/24536/5/be/src/exec/parquet/parquet-metadata-utils.cc@471 PS5, Line 471: chemaElement(const Colum > Not directly related to this change, but wouldn't col_type.IsPlainStringTyp IMO something like IsTextType() (true for string/char/varchar) would be the clearest -- To view, visit http://gerrit.cloudera.org:8080/24536 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I32d8c12a6b71708646fbfc06a6dd8f7aaf4f5e6b Gerrit-Change-Number: 24536 Gerrit-PatchSet: 12 Gerrit-Owner: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Arnab Karmakar <[email protected]> Gerrit-Reviewer: Balazs Hevele <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Mon, 21 Sep 2026 16:12:26 +0000 Gerrit-HasComments: Yes
