Tamas Mate has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/20883


Change subject: IMPALA-12706: Fix nested struct querying for Iceberg metadata 
tables
......................................................................

IMPALA-12706: Fix nested struct querying for Iceberg metadata tables

This commit fixes a DCHECK failure when querying a struct inside a
struct. The previous field accessor creation logic was trying to find
the ColumnDescriptor for a struct inside a struct and hit a DCHECK
because there are no ColumnDescriptors for struct fields. The logic
has been reworked to only use ColumnDescriptors for top level columns.

Testing:
 - Added E2E test to cover this case

Change-Id: Iadd029a4edc500bd8d8fca3f958903c2dbe09e8e
---
M be/src/exec/iceberg-metadata/iceberg-metadata-scan-node.cc
M be/src/exec/iceberg-metadata/iceberg-metadata-scan-node.h
M 
testdata/workloads/functional-query/queries/QueryTest/iceberg-metadata-tables.test
3 files changed, 29 insertions(+), 22 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iadd029a4edc500bd8d8fca3f958903c2dbe09e8e
Gerrit-Change-Number: 20883
Gerrit-PatchSet: 1
Gerrit-Owner: Tamas Mate <tma...@apache.org>

Reply via email to