----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13601/#review25221 -----------------------------------------------------------
Remove white spaces http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/builtin/AvroStorage.java <https://reviews.apache.org/r/13601/#comment49519> Can we remove this method? - Rohini Palaniswamy On Aug. 15, 2013, 7:09 p.m., Yuanli Dong Yuanli Dong wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13601/ > ----------------------------------------------------------- > > (Updated Aug. 15, 2013, 7:09 p.m.) > > > Review request for pig. > > > Repository: pig > > > Description > ------- > > Suppose I want to load data using this script: > a = load > './newavro/data/avro/Employee3.ser,./newavro/data/avro/Employee4.ser' USING > AvroStorage (); > It will fail because multiple paths separated by commas are not handled by > Avrostorage > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/builtin/AvroStorage.java > 1512538 > > http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/builtin/TrevniStorage.java > 1512538 > > http://svn.apache.org/repos/asf/pig/trunk/test/org/apache/pig/builtin/TestAvroStorage.java > 1512538 > > Diff: https://reviews.apache.org/r/13601/diff/ > > > Testing > ------- > > Test case added to TestAvroStorage class, passed after applying the patch. > > > Thanks, > > Yuanli Dong Yuanli Dong > >