You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by Daniel Dai <da...@gmail.com> on 2011/04/25 19:08:15 UTC

Review Request: BinStorage/PigStorageSchema cannot load data from a different namenode

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/658/
-----------------------------------------------------------

Review request for pig.


Summary
-------

See PIG-1865


This addresses bug PIG-1865.
    https://issues.apache.org/jira/browse/PIG-1865


Diffs
-----

  http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/builtin/BinStorage.java 1095143 

Diff: https://reviews.apache.org/r/658/diff


Testing
-------

Test-patch:
     [exec] -1 overall.
     [exec]
     [exec]     +1 @author.  The patch does not contain any @author tags.
     [exec]
     [exec]     -1 tests included.  The patch doesn't appear to include any new or modified tests.
     [exec]                         Please justify why no tests are needed for this patch.
     [exec]
     [exec]     +1 javadoc.  The javadoc tool did not generate any warning messages.
     [exec]
     [exec]     +1 javac.  The applied patch does not increase the total number of javac compiler warnings.
     [exec]
     [exec]     +1 findbugs.  The patch does not introduce any new Findbugs warnings.
     [exec]
     [exec]     +1 release audit.  The applied patch does not increase the total number of release audit warnings.

There is no test case added. It requires two clusters and is hard to do in unit test.

Unit-test:
    all pass

Manual-test:
    Tested using two clusters and BinStorage can access remote hdfs


Thanks,

Daniel