----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18332/ -----------------------------------------------------------
Review request for hive. Bugs: HIVE-6477 https://issues.apache.org/jira/browse/HIVE-6477 Repository: hive-git Description ------- Parquet uses ParquetByteInspector for tinyint. During UDAF's, Hive uses reflection on the parquet inspector to find a 'corresponding' inspector (JavaByteObjectInspector), resulting in ClassCastException. ParquetByteInspector needs to implement getPrimitiveJavaObject to return a byte, so that the corresponding inspector JavaObjectByteInspector can read it. Similar issue exists for ParquetShortInspector, which is used for smallint. Diffs ----- data/files/alltypesagg.txt PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/io/parquet/serde/primitive/ParquetByteInspector.java d5d1bf1 ql/src/java/org/apache/hadoop/hive/ql/io/parquet/serde/primitive/ParquetShortInspector.java 94f2813 ql/src/test/queries/clientpositive/parquet_agg.q PRE-CREATION ql/src/test/results/clientpositive/parquet_agg.q.out PRE-CREATION Diff: https://reviews.apache.org/r/18332/diff/ Testing ------- Added a unit test for the same. Thanks, Szehon Ho