----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11355/ -----------------------------------------------------------
(Updated June 4, 2013, 9:50 p.m.) Review request for pig and Rohini Palaniswamy. Changes ------- 1) Changed patch to use mock.Storage() 2) Smaller generated avro file Description ------- Patch to write default values to the Schema when the writer schema contains that in the AvroStorage. Diffs (updated) ----- http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/PigSchema2Avro.java 1485826 http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorage.java 1485826 Diff: https://reviews.apache.org/r/11355/diff/ Testing ------- Yes against the Piggybank in Pig trunk/Pig 0.12 Thanks, Viraj Bhat