> On June 3, 2013, 1:03 p.m., Rohini Palaniswamy wrote: > > Just minor comments in the naming of the variable. Java variable names > > should be camel case.
Thanks but now the verifyTxtResults method is not used any more - Viraj ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11333/#review21312 ----------------------------------------------------------- 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 > >
