drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "abdelhakim deneche" <adene...@gmail.com>
Subject Review Request 31783: DRILL-2220: Complex reader unable to read FIXED_LEN_BYTE_ARRAY types in parquet file
Date Fri, 06 Mar 2015 01:26:17 GMT

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

Review request for drill and Steven Phillips.


Bugs: DRILL-2220
    https://issues.apache.org/jira/browse/DRILL-2220


Repository: drill-git


Description
-------

this patch fixes the _DrillParquetReader_ to properly handle _FIXED_LEN_BYTE_ARRAY_ type,
and also to take into account the scale and precision of the field


Diffs
-----

  exec/java-exec/src/main/codegen/templates/AbstractFieldWriter.java 145130b 
  exec/java-exec/src/main/codegen/templates/BaseWriter.java 979d4ac 
  exec/java-exec/src/main/codegen/templates/MapWriters.java b8d5365 
  exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet2/DrillParquetGroupConverter.java
d219927 
  exec/java-exec/src/test/java/org/apache/drill/exec/store/parquet2/TestDrillParquetReader.java
PRE-CREATION 
  exec/java-exec/src/test/resources/parquet2/decimal28_38.parquet PRE-CREATION 

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


Testing
-------

added new unit test to make sure DECIMAL28/DECIMAL38 fields are read correctly when they are
OPTIONAL or REQUIRED
all unit tests are passing along with functional, customer and tpch100


Thanks,

abdelhakim deneche


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message