You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by Jason Altekruse <al...@gmail.com> on 2014/05/22 20:30:29 UTC

Review Request 21820: Drill - 649 - parquet dictionary support for fixed length types

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

Review request for drill.


Repository: drill-git


Description
-------

Fix for bug reading impala and avro generated files as they use the newer dictionary types by default.


Diffs
-----

  exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ColumnReader.java b9faafe 
  exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/NullableColumnReader.java 6040c67 
  exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/NullableFixedByteAlignedReaders.java PRE-CREATION 
  exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/PageReadStatus.java 021b622 
  exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetFixedWidthDictionaryReader.java PRE-CREATION 
  exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetRecordReader.java 9cdd205 
  exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/VarLenBinaryReader.java c217e80 
  exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/VarLengthColumnReaders.java f0f2146 
  exec/java-exec/src/test/java/org/apache/drill/exec/store/parquet/ParquetRecordReaderTest.java e594441 
  exec/java-exec/src/test/java/org/apache/drill/exec/store/parquet/ParquetResultListener.java 04197bc 

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


Testing
-------

added a new test to the parquet test file


Thanks,

Jason Altekruse