Csaba Ringhofer has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24567 )
Change subject: IMPALA-15158: Replace ColumnType.is_binary_ with an enum ...................................................................... IMPALA-15158: Replace ColumnType.is_binary_ with an enum This is preparation for adding new types that piggy-back on STRING (GEOMETRY/GEOGRAPHY). The new types will be available only in Parquet, so adding the actual types later should not need to modify unrelated file formats. Change-Id: I667138dac8afdd117dea8ed90a85927352f73269 Reviewed-on: http://gerrit.cloudera.org:8080/24567 Reviewed-by: Csaba Ringhofer <[email protected]> Tested-by: Csaba Ringhofer <[email protected]> --- M be/src/exec/hbase/hbase-table-writer.cc M be/src/exec/hdfs-text-table-writer.cc M be/src/exec/iceberg-metadata/iceberg-row-reader.cc M be/src/exec/kudu/kudu-util-ir.cc M be/src/exec/paimon/paimon-jni-row-reader.cc M be/src/exec/text-converter.inline.h M be/src/runtime/complex-value-writer.inline.h M be/src/runtime/types.cc M be/src/runtime/types.h 9 files changed, 136 insertions(+), 64 deletions(-) Approvals: Csaba Ringhofer: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/24567 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I667138dac8afdd117dea8ed90a85927352f73269 Gerrit-Change-Number: 24567 Gerrit-PatchSet: 9 Gerrit-Owner: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Balazs Hevele <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]>
