----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65478/ -----------------------------------------------------------
(Updated Feb. 13, 2018, 3:34 p.m.) Review request for hive. Repository: hive-git Description ------- VectorizedParquetReader throws an exception when trying to reading from a parquet table on which new columns are added. Diffs (updated) ----- ql/src/java/org/apache/hadoop/hive/ql/io/parquet/vector/BaseVectorizedColumnReader.java 907a9b8 ql/src/java/org/apache/hadoop/hive/ql/io/parquet/vector/ParquetDataColumnReader.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/io/parquet/vector/ParquetDataColumnReaderFactory.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/io/parquet/vector/VectorizedDummyColumnReader.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/io/parquet/vector/VectorizedListColumnReader.java c36640d ql/src/java/org/apache/hadoop/hive/ql/io/parquet/vector/VectorizedParquetRecordReader.java 08ac57b ql/src/java/org/apache/hadoop/hive/ql/io/parquet/vector/VectorizedPrimitiveColumnReader.java 39689f1 ql/src/java/org/apache/hadoop/hive/ql/io/parquet/vector/package-info.java PRE-CREATION ql/src/test/org/apache/hadoop/hive/ql/io/parquet/TestVectorizedColumnReader.java 9e414dc ql/src/test/org/apache/hadoop/hive/ql/io/parquet/TestVectorizedDictionaryEncodingColumnReader.java 3e5d831 ql/src/test/org/apache/hadoop/hive/ql/io/parquet/VectorizedColumnReaderTestBase.java 5d3ebd6 ql/src/test/queries/clientpositive/schema_evol_par_vec_table_dictionary_encoding.q PRE-CREATION ql/src/test/queries/clientpositive/schema_evol_par_vec_table_non_dictionary_encoding.q PRE-CREATION ql/src/test/results/clientpositive/schema_evol_par_vec_table.q.out PRE-CREATION ql/src/test/results/clientpositive/schema_evol_par_vec_table_dictionary_encoding.q.out PRE-CREATION ql/src/test/results/clientpositive/schema_evol_par_vec_table_non_dictionary_encoding.q.out PRE-CREATION Diff: https://reviews.apache.org/r/65478/diff/5/ Changes: https://reviews.apache.org/r/65478/diff/4-5/ Testing ------- Newly added UT passed and qtest passed locally. Thanks, cheng xu
