----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4281/ -----------------------------------------------------------
(Updated 2012-03-14 20:48:18.890073) Review request for hcatalog, Francis Liu, David Capwell, and Rohini Palaniswamy. Changes ------- -- Moved the setting of mapred output key and value class to FileOutputFormat, so that the updated props are available only before the record writer is created. -- Added a test case for text MR job. Summary ------- Creating a table stored a sequence file and trying to write data into it using Pig or MR fails with NPE. Diffs (updated) ----- /trunk/src/java/org/apache/hcatalog/mapreduce/FileOutputFormatContainer.java 1300711 /trunk/src/java/org/apache/hcatalog/mapreduce/FileRecordWriterContainer.java 1300711 /trunk/src/test/org/apache/hcatalog/mapreduce/TestSequenceFileReadWrite.java PRE-CREATION Diff: https://reviews.apache.org/r/4281/diff Testing ------- Unit tests added and they pass. Thanks, Vandana
