----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45128/#review125952 -----------------------------------------------------------
Ship it! Ship It! - Sergio Pena On March 21, 2016, 10:03 p.m., Jimmy Xiang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45128/ > ----------------------------------------------------------- > > (Updated March 21, 2016, 10:03 p.m.) > > > Review request for hive and Sergio Pena. > > > Bugs: HIVE-12619 > https://issues.apache.org/jira/browse/HIVE-12619 > > > Repository: hive-git > > > Description > ------- > > This patch makes sure if the order of fields in a struct is changed, the > projected type should have the right field order so that the query doesn't > fail. > > > Diffs > ----- > > > ql/src/java/org/apache/hadoop/hive/ql/io/parquet/convert/HiveSchemaConverter.java > b01f21f > > ql/src/java/org/apache/hadoop/hive/ql/io/parquet/read/DataWritableReadSupport.java > 53f3b72 > > ql/src/java/org/apache/hadoop/hive/ql/io/parquet/serde/ParquetHiveSerDe.java > e1bf8e2 > ql/src/test/queries/clientpositive/parquet_schema_evolution.q 193400f > ql/src/test/results/clientpositive/parquet_map_null.q.java1.8.out dd541a5 > ql/src/test/results/clientpositive/parquet_schema_evolution.q.out 0b88d84 > ql/src/test/results/clientpositive/parquet_type_promotion.q.out 55f9b27 > > Diff: https://reviews.apache.org/r/45128/diff/ > > > Testing > ------- > > unit test and test with my local cluster > > > Thanks, > > Jimmy Xiang > >
