Peter Rozsa has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/21931 )
Change subject: IMPALA-13205: Do not include Iceberg position fields for MERGE statements with INSERT merge clauses ...................................................................... IMPALA-13205: Do not include Iceberg position fields for MERGE statements with INSERT merge clauses This patch makes the Iceberg position field inclusion conditional by including them only if there are UPDATE or DELETE merge clauses that are listed in a MERGE statement. These fields can be omitted when there's no delete file creation at the sink of the MERGE statement. Tests: - iceberg-merge-insert-only planner test added Change-Id: Ib62c78dab557625fa86988559b3732591755106f --- M fe/src/main/java/org/apache/impala/analysis/IcebergMergeImpl.java M fe/src/test/java/org/apache/impala/planner/PlannerTest.java A testdata/workloads/functional-planner/queries/PlannerTest/iceberg-merge-insert-only.test 3 files changed, 378 insertions(+), 7 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/31/21931/4 -- To view, visit http://gerrit.cloudera.org:8080/21931 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ib62c78dab557625fa86988559b3732591755106f Gerrit-Change-Number: 21931 Gerrit-PatchSet: 4 Gerrit-Owner: Peter Rozsa <[email protected]> Gerrit-Reviewer: Daniel Becker <[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]>
