[email protected] has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/19223


Change subject: IMPALA-11711: Virtual columns should be skipped in 
'FileMetadataUtils::AddIcebergColumns'
......................................................................

IMPALA-11711: Virtual columns should be skipped in 
'FileMetadataUtils::AddIcebergColumns'

In the 'FileMetadataUtils::AddIcebergColumns' method, when the slot is
a virtual column, it should be skipped directly. That may affect that
when we query the Iceberg v2 table (the first column is a partition
column of bool type), wrong position-delete result is given.

Testing:
 - Add e2e tests
 - Locally tested the result of The Position-based Iceberg tables

Change-Id: I58faf3df6ae8a5bcabb1d2ac9f11a6fbcd74bc24
---
M be/src/exec/file-metadata-utils.cc
A 
testdata/workloads/functional-query/queries/QueryTest/iceberg-virtual-columns.test
M tests/query_test/test_iceberg.py
3 files changed, 44 insertions(+), 0 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I58faf3df6ae8a5bcabb1d2ac9f11a6fbcd74bc24
Gerrit-Change-Number: 19223
Gerrit-PatchSet: 1
Gerrit-Owner: Anonymous Coward <[email protected]>

Reply via email to