----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7053/#review12175 -----------------------------------------------------------
Ship it! +1 non-binding. The tests passed fine for H20 and H23 when I ran them for branch-0.10. It hangs in local mode when run with Y! distribution of pig using the -Dpig.dir option. I think we can address that in a separate jira. The patch does not apply cleanly on 0.10 due to PERL5LIB recent change. You will have to create one for 0.10. Also need to upload this patch in the jira. - Rohini Palaniswamy On Oct. 3, 2012, 12:53 p.m., Ivan Veselovsky wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7053/ > ----------------------------------------------------------- > > (Updated Oct. 3, 2012, 12:53 p.m.) > > > Review request for pig and Rohini Palaniswamy. > > > Description > ------- > > Please see https://issues.apache.org/jira/browse/PIG-2898 for details. > > > This addresses bug https://issues.apache.org/jira/browse/PIG-2898. > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/PIG-2898 > > > Diffs > ----- > > http://svn.apache.org/repos/asf/pig/trunk/test/e2e/harness/TestDriver.pm > 1393450 > http://svn.apache.org/repos/asf/pig/trunk/test/e2e/harness/test_harness.pl > 1393450 > http://svn.apache.org/repos/asf/pig/trunk/test/e2e/pig/build.xml 1393450 > http://svn.apache.org/repos/asf/pig/trunk/test/e2e/pig/conf/local.conf > 1393450 > > http://svn.apache.org/repos/asf/pig/trunk/test/e2e/pig/deployers/ExistingClusterDeployer.pm > 1393450 > > http://svn.apache.org/repos/asf/pig/trunk/test/e2e/pig/drivers/TestDriverPig.pm > 1393450 > > Diff: https://reviews.apache.org/r/7053/diff/ > > > Testing > ------- > > Tested e2e tests execution in both sequential (default) and parallel modes. > The test run duration measurement data (in dependency on the fork parameters) > will be available soon. > > > Thanks, > > Ivan Veselovsky > >
