[
https://issues.apache.org/jira/browse/IMPALA-13674?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17933118#comment-17933118
]
ASF subversion and git services commented on IMPALA-13674:
----------------------------------------------------------
Commit 167ced78448d73cdb4b742a3ba43b0752302a167 in impala's branch
refs/heads/master from Peter Rozsa
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=167ced784 ]
IMPALA-13674: Enable MERGE statement for Iceberg tables with equality deletes
This change fixes the delete expression calculation for
IcebergMergeImpl, when an Iceberg table contains equality deletes, the
merge implementation now includes the data sequence number in the result
expressions as the underlying tuple descriptor also includes it
implicitly. Without including this field, the row evaluation fails
because of the mismatching number of evaluators and slot descriptors.
Tests:
- manually validated on an Iceberg table that contains equality delete
- e2e test added
Change-Id: I60e48e2731a59520373dbb75104d75aae39a94c1
Reviewed-on: http://gerrit.cloudera.org:8080/22423
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
> MERGE statement targeting Iceberg tables with equality deletes runs on DCHECK
> -----------------------------------------------------------------------------
>
> Key: IMPALA-13674
> URL: https://issues.apache.org/jira/browse/IMPALA-13674
> Project: IMPALA
> Issue Type: Bug
> Reporter: Peter Rozsa
> Assignee: Peter Rozsa
> Priority: Major
> Labels: impala-iceberg
>
> Iceberg tables with equality deletes include one additional field implicitly
> (data sequence number), this causes invalid materialization in the MergeNode,
> as its tuple descriptor's size mismatches the size of the evaluators.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
