----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6884/#review12337 -----------------------------------------------------------
Ship it! I will run the unit tests and commit the patch if everything passes. - Santhosh Srinivasan On Oct. 10, 2012, 11:15 p.m., Cheolsoo Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6884/ > ----------------------------------------------------------- > > (Updated Oct. 10, 2012, 11:15 p.m.) > > > Review request for pig and Santhosh Srinivasan. > > > Description > ------- > > Add support for multiple avro schemas to AvroStorage. This patch is based on > Stan Rosenberg's original work. > > Please see https://issues.apache.org/jira/browse/PIG-2579 for details > > > This addresses bug PIG-2579. > https://issues.apache.org/jira/browse/PIG-2579 > > > Diffs > ----- > > > contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/AvroStorage.java > d7a004f > > contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/AvroStorageUtils.java > 84280af > > contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/PigAvroInputFormat.java > fb5cc25 > > contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/PigAvroRecordReader.java > 75057f9 > > contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorage.java > 1f6e581 > > contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorageUtils.java > 0761d5a > > Diff: https://reviews.apache.org/r/6884/diff/ > > > Testing > ------- > > New unit tests are added: > - TestAvroStorageUtils.testMergeSchema > - TestAvroStorage.testMultipleSchemas1,2 > > > Thanks, > > Cheolsoo Park > >
