Re: Review Request 62028: Remove Sqoop dependency on deprecated HBase APIs

2017-09-11 Thread Boglarka Egyed
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62028/#review185070 --- Ship it! Hi Szabolcs, Thanks to help preparing Sqoop for the

Re: Review Request 62028: Remove Sqoop dependency on deprecated HBase APIs

2017-09-11 Thread Zoltán Tóth
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62028/#review185068 --- Ship it! Ship It! - Zoltán Tóth On Sept. 11, 2017, 9:58

Re: Review Request 62028: Remove Sqoop dependency on deprecated HBase APIs

2017-09-11 Thread Szabolcs Vasas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62028/ --- (Updated Sept. 11, 2017, 9:58 a.m.) Review request for Sqoop. Bugs:

Re: Review Request 62028: Remove Sqoop dependency on deprecated HBase APIs

2017-09-11 Thread Szabolcs Vasas
> On Sept. 8, 2017, 2:14 p.m., Zoltán Tóth wrote: > > src/java/org/apache/sqoop/mapreduce/HBaseBulkImportJob.java > > Lines 98 (patched) > > > > > > Don't you want to log if it cannot get HBaseTable? Maybe it will be

Re: Review Request 62028: Remove Sqoop dependency on deprecated HBase APIs

2017-09-08 Thread Zoltán Tóth
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62028/#review184973 --- Hey Szabi, Thanks for the contribution especially for good test

Review Request 62028: Remove Sqoop dependency on deprecated HBase APIs

2017-09-01 Thread Szabolcs Vasas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62028/ --- Review request for Sqoop. Bugs: SQOOP-3232