Re: Review Request 69199: Create tests for SQOOP-2949, quote escaping in split-by

2018-10-31 Thread Boglarka Egyed
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69199/#review210223 --- Ship it! Hi Fero, Thanks for this fix! I ran unit tests succes

Re: Review Request 69199: Create tests for SQOOP-2949, quote escaping in split-by

2018-10-31 Thread Fero Szabo via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69199/ --- (Updated Oct. 31, 2018, 1:51 p.m.) Review request for Sqoop, Boglarka Egyed and

Re: Review Request 69199: Create tests for SQOOP-2949, quote escaping in split-by

2018-10-31 Thread Fero Szabo via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69199/ --- (Updated Oct. 31, 2018, 12:54 p.m.) Review request for Sqoop, Boglarka Egyed an

Re: Review Request 69199: Create tests for SQOOP-2949, quote escaping in split-by

2018-10-31 Thread Szabolcs Vasas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69199/#review210215 --- src/test/org/apache/sqoop/importjob/SplitByImportTest.java Lines

Re: Review Request 69199: Create tests for SQOOP-2949, quote escaping in split-by

2018-10-30 Thread Fero Szabo via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69199/ --- (Updated Oct. 30, 2018, 4:26 p.m.) Review request for Sqoop, Boglarka Egyed and