Hello Zoltan Borok-Nagy, Peter Rozsa, Noemi Pap-Takacs, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24994
to look at the new patch set (#3).
Change subject: IMPALA-13591: Fix NPE when dropping column used in old
partition spec
......................................................................
IMPALA-13591: Fix NPE when dropping column used in old partition spec
Dropping an Iceberg column that was previously used in a partition spec
(but no longer in the active spec) causes an NPE during table metadata
reload. convertPartitionSpec() assumes all partition field source columns
exist in the current schema, which fails after spec evolution + column
drop.
When findColumnName() returns null for a dropped source column, create a
VOID placeholder partition field instead of crashing. The BE already
skips VOID fields everywhere it iterates partition specs.
Testing: Added regression test in iceberg-alter-v2.test that creates a
table with bucket(7, p), evolves to bucket(7, i), drops column p, and
verifies both DDL success and data readability.
Change-Id: Ie5f1439b31d7f460008760909306607018ac094f
Assisted-by: Claude Opus 4.6 (Claude Code)
---
M fe/src/main/java/org/apache/impala/catalog/FeIcebergTable.java
M testdata/workloads/functional-query/queries/QueryTest/iceberg-alter-v2.test
2 files changed, 83 insertions(+), 5 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/94/24994/3
--
To view, visit http://gerrit.cloudera.org:8080/24994
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie5f1439b31d7f460008760909306607018ac094f
Gerrit-Change-Number: 24994
Gerrit-PatchSet: 3
Gerrit-Owner: Nandor Kollar <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
