-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19573/
-----------------------------------------------------------
(Updated March 26, 2014, 12:09 p.m.)
Review request for Tajo.
Bugs: TAJO-30
https://issues.apache.org/jira/browse/TAJO-30
Repository: tajo
Description (updated)
-------
TAJO-30: Add Parquet support.
Revision 2:
* Add missing property to tajo-storage/src/main/resource/storage-default.xml.
* Add TableStatistics to ParquetAppender
* Fix findbugs warnings
* Apply patch to add testNullHandlingTypes
* Add package-info.java documentation file that describes overview of Parquet
support and mapping between Parquet and Tajo data types.
* Fix the way parquet-tajo handles NULLs
* Clean up and add more Javadoc comments
* Add test for TajoSchemaConverter
Diffs
-----
tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java
5ad2ecaca89604f133d288fe44daa14716ac82f2
tajo-catalog/tajo-catalog-common/src/main/proto/CatalogProtos.proto
14fb39f54bbba756d9d8a048055a177a79cbe6ff
tajo-storage/pom.xml 5521ddc7868342146d8619e45cc6e55c4c2eff53
tajo-storage/src/main/java/org/apache/tajo/storage/parquet/ParquetAppender.java
PRE-CREATION
tajo-storage/src/main/java/org/apache/tajo/storage/parquet/ParquetScanner.java
PRE-CREATION
tajo-storage/src/main/java/org/apache/tajo/storage/parquet/TajoParquetReader.java
PRE-CREATION
tajo-storage/src/main/java/org/apache/tajo/storage/parquet/TajoParquetWriter.java
PRE-CREATION
tajo-storage/src/main/java/org/apache/tajo/storage/parquet/TajoReadSupport.java
PRE-CREATION
tajo-storage/src/main/java/org/apache/tajo/storage/parquet/TajoRecordConverter.java
PRE-CREATION
tajo-storage/src/main/java/org/apache/tajo/storage/parquet/TajoRecordMaterializer.java
PRE-CREATION
tajo-storage/src/main/java/org/apache/tajo/storage/parquet/TajoSchemaConverter.java
PRE-CREATION
tajo-storage/src/main/java/org/apache/tajo/storage/parquet/TajoWriteSupport.java
PRE-CREATION
tajo-storage/src/main/java/org/apache/tajo/storage/parquet/package-info.java
PRE-CREATION
tajo-storage/src/main/resources/storage-default.xml
5bf4453f108652f7e83be4a91333f867687670bc
tajo-storage/src/test/java/org/apache/tajo/storage/TestMergeScanner.java
f2a66d92b084d9ac2eb7a912cc45a7b5fc8d56e8
tajo-storage/src/test/java/org/apache/tajo/storage/TestStorages.java
1b36b7843d6a34805a5f6d328726b6244f1ac59a
tajo-storage/src/test/java/org/apache/tajo/storage/parquet/TestReadWrite.java
PRE-CREATION
tajo-storage/src/test/java/org/apache/tajo/storage/parquet/TestSchemaConverter.java
PRE-CREATION
tajo-storage/src/test/java/org/apache/tajo/storage/v2/TestStorages.java
05be40a5d22fe52ba555a1e6b57a0b33a57a7b53
tajo-storage/src/test/resources/storage-default.xml
304af10f6d165432b86f1d9846339122226bc59d
Diff: https://reviews.apache.org/r/19573/diff/
Testing (updated)
-------
Added unit test for reading and writing to Parquet and unit tests for
TajoSchemaConverter. Added Parquet to storage tests. All tests pass.
Thanks,
David Chen
