Pooja Nilangekar has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/10704


Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet 
scans
......................................................................

IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

This change ensures that the planner computes parquet conjuncts
only for scans containing parquet files. Additionally, it also
handles PARQUET_DICTIONARY_FILTERING and PARQUET_READ_STATISTICS
query options in the planner.

Testing was carried out independently on parquet and non-parquet
scans:
  1. Parquet scans were tested via the existing parquet-filtering
     planner test. Additionally, a new test
     [parquet-filtering-disabled] was added to ensure that the
     explain plan generated skips parquet predicates based on the
     query options.
  2. Non-parquet scans were tested manually to ensure that the
     functions to compute parquet conjuncts were not invoked.
     This was done because the eventual explain plan returned to
     the client doesn't contain parquet predicates for non-parquet
     scans.

Change-Id: I9d6c26d42db090c8a15c602f6419ad6399c329e7
---
M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java
M fe/src/test/java/org/apache/impala/planner/PlannerTest.java
A 
testdata/workloads/functional-planner/queries/PlannerTest/parquet-filtering-disabled.test
3 files changed, 296 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/04/10704/2
--
To view, visit http://gerrit.cloudera.org:8080/10704
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9d6c26d42db090c8a15c602f6419ad6399c329e7
Gerrit-Change-Number: 10704
Gerrit-PatchSet: 2
Gerrit-Owner: Pooja Nilangekar <pooja.nilange...@cloudera.com>

Reply via email to