Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2017-01-09 Thread Adam Szita
> On Dec. 13, 2016, 6:22 p.m., Rohini Palaniswamy wrote: > > ivy/libraries.properties, lines 75-76 > > > > > > Having two versions of netty will actually create conflict. Hadoop > > folks were talking about this in r

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2017-01-06 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54056/#review160784 --- Ship it! Ship It! - Rohini Palaniswamy On Dec. 19, 2016, 9:2

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2017-01-06 Thread Rohini Palaniswamy
> On Dec. 13, 2016, 6:22 p.m., Rohini Palaniswamy wrote: > > ivy/libraries.properties, lines 75-76 > > > > > > Having two versions of netty will actually create conflict. Hadoop > > folks were talking about this in r

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-12-19 Thread Adam Szita
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54056/ --- (Updated Dec. 19, 2016, 9:25 a.m.) Review request for pig, Daniel Dai and Rohin

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-12-16 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54056/#review159515 --- build.xml (lines 384 - 388)

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-12-15 Thread Adam Szita
> On Dec. 13, 2016, 6:22 p.m., Rohini Palaniswamy wrote: > > ivy/libraries.properties, lines 75-76 > > > > > > Having two versions of netty will actually create conflict. Hadoop > > folks were talking about this in r

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-12-15 Thread Adam Szita
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54056/ --- (Updated Dec. 15, 2016, 1:02 p.m.) Review request for pig, Daniel Dai and Rohin

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-12-13 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54056/#review158925 --- build.xml (lines 903 - 904)

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-12-13 Thread Adam Szita
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54056/ --- (Updated Dec. 13, 2016, 10:53 a.m.) Review request for pig, Daniel Dai and Rohi

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-12-12 Thread Rohini Palaniswamy
> On Dec. 6, 2016, 1:17 p.m., Adam Szita wrote: > > src/org/apache/pig/builtin/TextLoader.java, line 262 > > > > > > Does this mean it wasn't working properly ever with hadoopversion=23? > > > > In Hadoop23

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-12-06 Thread Adam Szita
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54056/#review158152 --- src/org/apache/pig/builtin/TextLoader.java

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-12-06 Thread Adam Szita
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54056/ --- (Updated Dec. 6, 2016, 12:55 p.m.) Review request for pig, Daniel Dai and Rohin

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-12-06 Thread Adam Szita
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54056/ --- (Updated Dec. 6, 2016, 10:16 a.m.) Review request for pig, Daniel Dai and Rohin

Re: Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-12-02 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54056/#review157618 --- Comments: 1) Can you change hadoopversion 23 to 2 everywhere. W

Review Request 54056: Drop Hadoop 1.x support in Pig 0.17

2016-11-24 Thread Adam Szita
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54056/ --- Review request for pig, Daniel Dai and Rohini Palaniswamy. Bugs: PIG-4923 h