> On June 2, 2013, 9:27 p.m., Cheolsoo Park wrote: > > http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorage.java, > > line 1104 > > <https://reviews.apache.org/r/11333/diff/5/?file=298357#file298357line1104> > > > > If you use mock.Storage here instead of PigStoage, you won't need the > > verifyTextResults method and extra output file. Can you please update your > > test? > > > > Please see org.apache.pig.builtin.mock.Storage.java.
Added Mock Storage - Viraj ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11333/#review21305 ----------------------------------------------------------- On June 4, 2013, 12:15 a.m., Viraj Bhat wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11333/ > ----------------------------------------------------------- > > (Updated June 4, 2013, 12:15 a.m.) > > > Review request for pig and Rohini Palaniswamy. > > > Description > ------- > > Null pointer exception when loading union with null in it's schema. Test case > was also updated with a sample test case. > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/AvroStorage.java > 1485358 > > http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/PigAvroRecordReader.java > 1485358 > > http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorage.java > 1485358 > > Diff: https://reviews.apache.org/r/11333/diff/ > > > Testing > ------- > > Yes all tests pass in the piggybank > > > Thanks, > > Viraj Bhat > >
