-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36942/
-----------------------------------------------------------

Review request for hive and Aihua Xu.


Bugs: HIVE-11401
    https://issues.apache.org/jira/browse/HIVE-11401


Repository: hive-git


Description
-------

The following patch reviews the predicate created by Hive, and removes any 
column that does not belong to the Parquet schema, such as partitioned columns. 
This way Parquet can filter the columns correctly.


Diffs
-----

  
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/read/ParquetFilterPredicateConverter.java
 PRE-CREATION 
  
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/read/ParquetRecordReaderWrapper.java
 49e52da2e26fd7213df1db88716eaee94cb536b8 
  
ql/src/test/org/apache/hadoop/hive/ql/io/parquet/TestParquetRecordReaderWrapper.java
 87dd344534f09c7fc565fdc467ac82a51f37ebba 
  
ql/src/test/org/apache/hadoop/hive/ql/io/parquet/read/TestParquetFilterPredicate.java
 PRE-CREATION 
  ql/src/test/org/apache/hadoop/hive/ql/io/sarg/TestConvertAstToSearchArg.java 
85e952fb6855a2a03902ed971f54191837b32dac 
  ql/src/test/queries/clientpositive/parquet_predicate_pushdown.q PRE-CREATION 
  ql/src/test/results/clientpositive/parquet_predicate_pushdown.q.out 
PRE-CREATION 

Diff: https://reviews.apache.org/r/36942/diff/


Testing
-------

Unit tests: TestParquetFilterPredicate.java
Integration tests: parquet_predicate_pushdown.q


Thanks,

Sergio Pena

Reply via email to