-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59020/
-----------------------------------------------------------
(Updated July 12, 2017, 2:53 p.m.)
Review request for hive, Aihua Xu and Sergio Pena.
Changes
-------
Attached rebased patch (due to timestamp issue revert) + addressed comments
Bugs: HIVE-8838
https://issues.apache.org/jira/browse/HIVE-8838
Repository: hive-git
Description
-------
Adding support for HCatalog to write tables stored in Parquet format
Diffs (updated)
-----
hcatalog/core/src/main/java/org/apache/hive/hcatalog/mapreduce/FileRecordWriterContainer.java
b2abc5fbb3670893415354552239d67d072459ed
hcatalog/core/src/main/java/org/apache/hive/hcatalog/mapreduce/SpecialCases.java
60af5c0bf397273fb820f0ee31e578745dbc200f
hcatalog/hcatalog-pig-adapter/src/test/java/org/apache/hive/hcatalog/pig/TestHCatLoaderComplexSchema.java
4c686fec596d39d41d458bc3ea2753877bd9df98
hcatalog/hcatalog-pig-adapter/src/test/java/org/apache/hive/hcatalog/pig/TestHCatLoaderEncryption.java
ad11eab1b7e67541b56e90e4a85ba37b41a4db92
hcatalog/hcatalog-pig-adapter/src/test/java/org/apache/hive/hcatalog/pig/TestHCatStorerMulti.java
918332ddfda58306707d326f8668b2c223110a29
hcatalog/hcatalog-pig-adapter/src/test/java/org/apache/hive/hcatalog/pig/TestParquetHCatLoader.java
6cd382145b55d6b85fc3366faeaba2aaef65ab04
hcatalog/hcatalog-pig-adapter/src/test/java/org/apache/hive/hcatalog/pig/TestParquetHCatStorer.java
6dfdc04954dd0b110b1a7194e69468b5dc2f842e
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/MapredParquetOutputFormat.java
379a9135d9c631b2f473976b00f3dc87f9fec0c4
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/serde/ParquetTableUtils.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/write/ParquetRecordWriterWrapper.java
c021dafa480e65d7c0c19a5a85988464112468cb
ql/src/test/org/apache/hadoop/hive/ql/io/parquet/TestMapredParquetOutputFormat.java
ec85b5df0f95cbd45b87259346ae9c1e5aa604a4
Diff: https://reviews.apache.org/r/59020/diff/2/
Changes: https://reviews.apache.org/r/59020/diff/1-2/
Testing
-------
Tested on cluster, and re-enabled previously disabled tests in HCatalog (for
Parquet) that were failing (this adds ~40 tests to be run)
Thanks,
Adam Szita
