Xuebin Su has uploaded a new patch set (#3). ( 
http://gerrit.cloudera.org:8080/22974 )

Change subject: IMPALA-14110: Avoid decoding values for counting columns
......................................................................

IMPALA-14110: Avoid decoding values for counting columns

For a counting column, its slot descriptor is null and its data decoder
is not initialized. Therefore, trying to decode the values when skipping
them will lead to check failure.

This patch fixes the issue by returning early when trying to skip values
if the current column is a counting column to avoid trying to decode any
value.

Testing:
- Passed TestZippingUnnest in exhaustive mode.
- Added a test case to make sure that page filtering works for counting
  columns.

Change-Id: Ia707335c50cc0653097f375aae3f10609e0eb091
---
M be/src/exec/parquet/hdfs-parquet-scanner.cc
M be/src/exec/parquet/parquet-column-readers.cc
M 
testdata/workloads/functional-query/queries/QueryTest/nested-types-parquet-page-index.test
M 
testdata/workloads/functional-query/queries/QueryTest/zipping-unnest-in-from-clause.test
4 files changed, 19 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/74/22974/3
--
To view, visit http://gerrit.cloudera.org:8080/22974
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia707335c50cc0653097f375aae3f10609e0eb091
Gerrit-Change-Number: 22974
Gerrit-PatchSet: 3
Gerrit-Owner: Xuebin Su <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>

Reply via email to