-----------------------------------------------------------
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 (updated)
-----

  
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

Reply via email to