----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5936/#review9233 -----------------------------------------------------------
Needs more test cases to increase code coverage. There are no negative tests. contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/AvroStorageUtils.java <https://reviews.apache.org/r/5936/#comment19778> Need a test case to cover this code path contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/AvroStorageUtils.java <https://reviews.apache.org/r/5936/#comment19779> Can you add a test case to cover this code path? contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorage.java <https://reviews.apache.org/r/5936/#comment19777> What if you have test_glob{3,2,1}.avro - will the test case fail? - Santhosh Srinivasan On July 15, 2012, 2:51 a.m., Cheolsoo Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5936/ > ----------------------------------------------------------- > > (Updated July 15, 2012, 2:51 a.m.) > > > Review request for pig. > > > Description > ------- > > Add glob support to AvroStorage: > > https://issues.apache.org/jira/browse/PIG-2492 > > > This addresses bug PIG-2492. > https://issues.apache.org/jira/browse/PIG-2492 > > > Diffs > ----- > > > contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/AvroStorage.java > 0f8ef27 > > contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/AvroStorageUtils.java > c7de726 > > contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorage.java > 48b093b > > Diff: https://reviews.apache.org/r/5936/diff/ > > > Testing > ------- > > 1. Added new unit tests as follows: > > - testDir verifies that AvroStorage recursively loads files in a directory > and its sub-directories. > - testGlob1 to 3 verify that glob patterns are expanded properly. > > To run the tests, please do the following: > > wget > https://issues.apache.org/jira/secure/attachment/12536534/avro_test_files.tar.gz > > tar -xf avro_test_files.tar.gz > ant clean compile-test piggybank -Dhadoopversion=20 > cd contrib/piggybank/java > ant test -Dtestcase=TestAvroStorage > > 2. Both TestAvroStorage and TestAvroStorageUtils pass. > > > Thanks, > > Cheolsoo Park > >
