Joe McDonnell has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/22666


Change subject: IMPALA-13882: Fix Iceberg v2 deletes with tuple caching
......................................................................

IMPALA-13882: Fix Iceberg v2 deletes with tuple caching

A variety of Iceberg statements (including v2 deletes) rely
on getting information from the scan node child of the
delete node. Since tuple caching can insert a TupleCacheNode
above that scan, the logic is currently failing, because
it doesn't know how to bypass the TupleCacheNode and get
to the scan node below.

This modifies the logic in multiple places to detect a
TupleCacheNode and go past it to the get the scan node
below it.

Testing:
 - Added a basic Iceberg test with v2 deletes for the
  frontend test and custom cluster test

Change-Id: I162e738c4e4449a536701a740272aaac56ce8fd8
---
M be/src/exec/iceberg-delete-builder.cc
M be/src/runtime/fragment-state.cc
M fe/src/main/java/org/apache/impala/planner/IcebergDeleteNode.java
M fe/src/test/java/org/apache/impala/planner/TupleCacheTest.java
M tests/custom_cluster/test_tuple_cache.py
5 files changed, 58 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/66/22666/1
--
To view, visit http://gerrit.cloudera.org:8080/22666
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I162e738c4e4449a536701a740272aaac56ce8fd8
Gerrit-Change-Number: 22666
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com>

Reply via email to