Re: Review Request: PIG-2898: allow to run pig e2e tests in parallel mode.

2012-10-25 Thread Ivan Veselovsky
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7053/ --- (Updated Oct. 25, 2012, 5:21 p.m.) Review request for pig and Rohini Palaniswamy

Re: Review Request: PIG-2898: allow to run pig e2e tests in parallel mode.

2012-10-04 Thread Rohini Palaniswamy
> On Oct. 4, 2012, 5:55 p.m., Rohini Palaniswamy wrote: > > +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

Re: Review Request: PIG-2898: allow to run pig e2e tests in parallel mode.

2012-10-04 Thread Rohini Palaniswamy
--- 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 whe

Re: Review Request: PIG-2898: allow to run pig e2e tests in parallel mode.

2012-10-03 Thread Ivan Veselovsky
--- 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

Re: Review Request: PIG-2898: allow to run pig e2e tests in parallel mode.

2012-10-03 Thread Ivan Veselovsky
> On Oct. 2, 2012, 6:23 p.m., Rohini Palaniswamy wrote: > > http://svn.apache.org/repos/asf/pig/trunk/test/e2e/harness/TestDriver.pm, > > lines 89-95 > > > > > > Any reason for this length based formatting? In fact it m

Re: Review Request: PIG-2898: allow to run pig e2e tests in parallel mode.

2012-10-02 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7053/#review12079 --- Just few minor comments. I ran the patch on branch-0.10 after rever

Re: Review Request: PIG-2898: allow to run pig e2e tests in parallel mode.

2012-09-28 Thread Ivan Veselovsky
> On Sept. 20, 2012, 9:06 p.m., Rohini Palaniswamy wrote: > > http://svn.apache.org/repos/asf/pig/trunk/test/e2e/pig/drivers/TestDriverPig.pm, > > lines 136-154 > > > > > > This was taking 40 secs for each group in par

Re: Review Request: PIG-2898: allow to run pig e2e tests in parallel mode.

2012-09-28 Thread Ivan Veselovsky
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7053/ --- (Updated Sept. 28, 2012, 12:20 p.m.) Review request for pig and Rohini Palaniswa

Re: Review Request: PIG-2898: allow to run pig e2e tests in parallel mode.

2012-09-20 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7053/#review11749 --- General comments: 1) It would be good to rename file.fork.factor to f

Review Request: PIG-2898: allow to run pig e2e tests in parallel mode.

2012-09-12 Thread Ivan Veselovsky
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7053/ --- Review request for pig and Rohini Palaniswamy. Description --- Please see h