Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24994 )
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) Reviewed-on: http://gerrit.cloudera.org:8080/24994 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/catalog/FeIcebergTable.java M testdata/workloads/functional-query/queries/QueryTest/iceberg-alter-v2.test M tests/query_test/test_iceberg.py 3 files changed, 116 insertions(+), 5 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: Ie5f1439b31d7f460008760909306607018ac094f Gerrit-Change-Number: 24994 Gerrit-PatchSet: 6 Gerrit-Owner: Nandor Kollar <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Nandor Kollar <[email protected]> Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
