----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11135/ -----------------------------------------------------------
(Updated June 13, 2013, 6:34 p.m.) Review request for pig and Rohini Palaniswamy. Changes ------- 1) Change the testcase to use mockstorage 2) Remove the condition that does not verify results in Hadoop 23 3) Add back the "usemultipleSchemas" flag to handle cases when schemaToMergedSchemaMap is null and multiple_schemas is invoked. Test case testMultipleSchema1 fails for the previous patch 4) Testing done with Hadoop 23 Description ------- Default values are not honoured when merging default schema This addresses bug PIG-3318. https://issues.apache.org/jira/browse/PIG-3318 Diffs (updated) ----- http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/AvroStorage.java 1491556 http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/AvroStorageUtils.java 1491556 http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/PigAvroInputFormat.java 1491556 http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/PigAvroRecordReader.java 1491556 http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorage.java 1491562 Diff: https://reviews.apache.org/r/11135/diff/ Testing ------- Yes Thanks, Viraj Bhat