Re: Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-17 Thread Fero Szabo via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65017/#review195572 --- Ship it! Hi Szabi, Thanks for removing these deprecated classe

Re: Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-12 Thread Boglarka Egyed
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65017/#review195323 --- Ship it! Hi Szabi, Thanks for correcting my findings on the pu

Re: Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-11 Thread Anna Szonyi via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65017/#review195227 --- Ship it! Hi Szabi, This is great, it helps a lot with the read

Re: Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-09 Thread Szabolcs Vasas
> On Jan. 9, 2018, 1:46 p.m., Attila Szabo wrote: > > IMHO build.xml line 1098 (after this patch applied) should be also removed > > (this is an entry for the src/com/cloudera/sqoop directory to be included > > in the javadoc generation). > > > > Otherwise the patch looks okay, but please let

Re: Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-09 Thread Szabolcs Vasas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65017/ --- (Updated Jan. 9, 2018, 2:25 p.m.) Review request for Sqoop. Changes ---

Re: Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-09 Thread Attila Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65017/#review195030 --- IMHO build.xml line 1098 (after this patch applied) should be also

Re: Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-09 Thread Szabolcs Vasas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65017/ --- (Updated Jan. 9, 2018, 12:52 p.m.) Review request for Sqoop. Changes ---

Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-08 Thread Szabolcs Vasas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65017/ --- Review request for Sqoop. Bugs: SQOOP-3273 https://issues.apache.org/jira/b