-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4281/
-----------------------------------------------------------

(Updated 2012-03-13 22:16:02.719615)


Review request for hcatalog, Francis Liu, David Capwell, and Rohini Palaniswamy.


Changes
-------

-- Added code to set the key class to be NullWritable.
-- Added code to set the value class to be obtained from the serde in the 
foster storage handler. 
-- Added unit test which has a MR write job to a sequence file based HCat table 
and read the written data. 


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/FileRecordWriterContainer.java 
1300370 
  /trunk/src/java/org/apache/hcatalog/mapreduce/FosterStorageHandler.java 
1300370 
  /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

Reply via email to