-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1861/
-----------------------------------------------------------
(Updated 2011-09-29 22:35:16.427726)
Review request for hcatalog and Sushanth Sowmyan.
Changes
-------
Added javadoc comments to new classes and new un-overriden methods. Added
"Container" suffix to container class implementations.
Summary
-------
review of HCatalog-60, refactor HCatalog output stuff to support non file-based
storage drivers
This addresses bug HCatalog-60.
https://issues.apache.org/jira/browse/HCatalog-60
Diffs (updated)
-----
src/java/org/apache/hcatalog/mapreduce/DefaultOutputCommitterContainer.java
PRE-CREATION
src/java/org/apache/hcatalog/mapreduce/DefaultOutputFormatContainer.java
PRE-CREATION
src/java/org/apache/hcatalog/mapreduce/DefaultRecordWriterContainer.java
PRE-CREATION
src/java/org/apache/hcatalog/mapreduce/FileOutputCommitterContainer.java
PRE-CREATION
src/java/org/apache/hcatalog/mapreduce/FileOutputFormatContainer.java
PRE-CREATION
src/java/org/apache/hcatalog/mapreduce/FileOutputStorageDriver.java
PRE-CREATION
src/java/org/apache/hcatalog/mapreduce/FileRecordWriterContainer.java
PRE-CREATION
src/java/org/apache/hcatalog/mapreduce/HCatBaseOutputCommitter.java 8516e1a
src/java/org/apache/hcatalog/mapreduce/HCatBaseOutputFormat.java d289714
src/java/org/apache/hcatalog/mapreduce/HCatEximOutputCommitter.java 4e5c4d1
src/java/org/apache/hcatalog/mapreduce/HCatEximOutputFormat.java 47ddae3
src/java/org/apache/hcatalog/mapreduce/HCatOutputCommitter.java 973a4b6
src/java/org/apache/hcatalog/mapreduce/HCatOutputFormat.java 7a99caa
src/java/org/apache/hcatalog/mapreduce/HCatOutputStorageDriver.java 8ef0d25
src/java/org/apache/hcatalog/mapreduce/HCatRecordWriter.java e71b9af
src/java/org/apache/hcatalog/mapreduce/OutputCommitterContainer.java
PRE-CREATION
src/java/org/apache/hcatalog/mapreduce/OutputFormatContainer.java
PRE-CREATION
src/java/org/apache/hcatalog/mapreduce/RecordWriterContainer.java
PRE-CREATION
src/java/org/apache/hcatalog/pig/HCatEximStorer.java 3f1db06
src/java/org/apache/hcatalog/pig/HCatStorer.java 90d14c0
src/java/org/apache/hcatalog/rcfile/RCFileOutputDriver.java 6efc4d9
src/test/org/apache/hcatalog/mapreduce/HCatMapReduceTest.java 2dee111
src/test/org/apache/hcatalog/mapreduce/TestHCatDynamicPartitioned.java
dfc7a53
src/test/org/apache/hcatalog/mapreduce/TestHCatOutputFormat.java 506ed4e
src/test/org/apache/hcatalog/rcfile/TestRCFileOutputStorageDriver.java
4c4acfe
Diff: https://reviews.apache.org/r/1861/diff
Testing
-------
all unit tests passed
Thanks,
Francis
