Csaba Ringhofer has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15598
Change subject: IMPALA-9572: Fix DCHECK in nested Parquet scanning ...................................................................... IMPALA-9572: Fix DCHECK in nested Parquet scanning The issue occurred when there were skipped pages and a column inside a collection was scanned, but its position was not needed. The repetition level still needs to be read in this case, as the skipped ranges are set in top level rows, so collection items need to know which top level row do they belong to. A DCHECK in StrideWriter's constructor was hit, otherwise the code ran correctly in release mode. The DCHECK is moved to functions where the condition would actually cause problems. Testing: - added and ran a regression test Change-Id: I5e8ef514ead71f732c73f910af7fd1aecd37bb81 --- M be/src/exec/parquet/parquet-column-readers.cc M be/src/util/mem-util.h M testdata/workloads/functional-query/queries/QueryTest/nested-types-parquet-page-index.test 3 files changed, 18 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/98/15598/1 -- To view, visit http://gerrit.cloudera.org:8080/15598 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I5e8ef514ead71f732c73f910af7fd1aecd37bb81 Gerrit-Change-Number: 15598 Gerrit-PatchSet: 1 Gerrit-Owner: Csaba Ringhofer <csringho...@cloudera.com>
