Hello Tamas Mate, wangsheng, Csaba Ringhofer, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/18102

to look at the new patch set (#3).

Change subject: IMPALA-11051: Add support for 'void' Iceberg partition transform
......................................................................

IMPALA-11051: Add support for 'void' Iceberg partition transform

Iceberg recently added a new partition transform called 'void':
https://iceberg.apache.org/#spec/#partition-transforms

This patch adds support for this transform.

When the user wants to drop a column from the partition spec,
the VOID transform should be used instead of just omitting
the column. Simply omitting the column might cause problems when
the metadata table is being queried (currently only supported
by other engines).

Testing
 * added SHOW CREATE TABLE test
 * added e2e test

Change-Id: Icbe11d56cdeb82aaadedfdb3ad61dd7cc4c2f4d0
---
M be/src/exec/hdfs-table-sink.cc
M be/src/runtime/descriptors.cc
M be/src/runtime/descriptors.h
M common/thrift/CatalogObjects.thrift
M fe/src/main/java/org/apache/impala/analysis/InsertStmt.java
M fe/src/main/java/org/apache/impala/util/IcebergUtil.java
M 
testdata/workloads/functional-query/queries/QueryTest/iceberg-partitioned-insert.test
M testdata/workloads/functional-query/queries/QueryTest/show-create-table.test
8 files changed, 155 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/02/18102/3
--
To view, visit http://gerrit.cloudera.org:8080/18102
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Icbe11d56cdeb82aaadedfdb3ad61dd7cc4c2f4d0
Gerrit-Change-Number: 18102
Gerrit-PatchSet: 3
Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tamas Mate <[email protected]>
Gerrit-Reviewer: wangsheng <[email protected]>

Reply via email to