Re: Review Request 64371: SQOOP-3233: SqoopHCatImportHelper.convertNumberTypes check for Varchar instead of Char

2017-12-08 Thread Szabolcs Vasas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64371/#review193237 --- Ship it! Hi Feró, Thank you for the fix, I have ran the unit

Re: Review Request 64371: SQOOP-3233: SqoopHCatImportHelper.convertNumberTypes check for Varchar instead of Char

2017-12-07 Thread Fero Szabo via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64371/ --- (Updated Dec. 7, 2017, 5:23 p.m.) Review request for Sqoop, Boglarka Egyed and

Re: Review Request 64371: SQOOP-3233: SqoopHCatImportHelper.convertNumberTypes check for Varchar instead of Char

2017-12-07 Thread Szabolcs Vasas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64371/#review193133 ---

Re: Review Request 64371: SQOOP-3233: SqoopHCatImportHelper.convertNumberTypes check for Varchar instead of Char

2017-12-06 Thread Boglarka Egyed
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64371/#review192997 --- Ship it! Hi Fero, Thanks for fixing the patch file. Your

Re: Review Request 64371: SQOOP-3233: SqoopHCatImportHelper.convertNumberTypes check for Varchar instead of Char

2017-12-06 Thread Boglarka Egyed
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64371/#review192985 --- Hi Fero, I think some problem occured during patch generation as

Re: Review Request 64371: SQOOP-3233: SqoopHCatImportHelper.convertNumberTypes check for Varchar instead of Char

2017-12-06 Thread Fero Szabo via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64371/ --- (Updated Dec. 6, 2017, 2:13 p.m.) Review request for Sqoop, Boglarka Egyed and