----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7676/#review12694 -----------------------------------------------------------
Ship it! Ship It! - Bill Graham On Oct. 23, 2012, 4:48 p.m., Cheolsoo Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7676/ > ----------------------------------------------------------- > > (Updated Oct. 23, 2012, 4:48 p.m.) > > > Review request for pig and Santhosh Srinivasan. > > > Description > ------- > > The changes include: > > 1. Stop bundling hbase.jar and zookeoper.jar with pig.jar. So there should be > no longer incompatibility issues when using pig.jar with different versions > of hbase.jar. But to use HBaseStorage, HBASE_HOME and ZOOKEEPER_HOME must be > set by the user. Note that I am adding protobuf-java.jar to pig.jar because > otherwise it has to be explicitly added to PIG_CLASSPATH to use HBaseStorage, > which is not very intuitive. > > 2. Bump hbase and zk to 0.94.1 and 3.4.3 respectively. Since we no longer > bundle them in pig.jar, which versions we use doesn't matter. These jar files > will be used for unit test only. > > 3. Make the unit test cases work with newer versions of hbase and zk. > > 4. Add hbase runtime dependencies to ivy.xml. > > > This addresses bug PIG-2885. > https://issues.apache.org/jira/browse/PIG-2885 > > > Diffs > ----- > > build.xml 6b04f8a > ivy.xml 6e0a2e5 > ivy/libraries.properties 55da6c6 > test/org/apache/pig/test/TestHBaseStorage.java cc1efef > test/org/apache/pig/test/TestJobSubmission.java 021662f > > Diff: https://reviews.apache.org/r/7676/diff/ > > > Testing > ------- > > ant clean test-commit -Dhadoopversion=20 > ant clean test -Dtestcase=TestHBaseStorage -Dhadoopversion=20 > ant clean test -Dtestcase=TestJobSumission -Dhadoopversion=20 > > I also manually tested pig.jar with hbase 0.90 and 0.94. Once HBASE_HOME and > ZOOKEEPER_HOME are set, HBaseStorage works fine with both versions. > > > Thanks, > > Cheolsoo Park > >
