Re: Review Request 37923: SQOOP-2547 Sqoop2: RESTiliency: Provide REST tests for v1/connector servlet

2015-09-09 Thread Jarek Cecho
> On Sept. 8, 2015, 5:39 a.m., Dian Fu wrote: > > test/src/test/java/org/apache/sqoop/integration/server/InvalidRESTCallsTest.java, > > line 148 > > > > > > Patch ofSQOOP-2544 has removed the check of the request

Re: Review Request 37923: SQOOP-2547 Sqoop2: RESTiliency: Provide REST tests for v1/connector servlet

2015-09-08 Thread Jarek Cecho
> On Sept. 8, 2015, 5:39 a.m., Dian Fu wrote: > > test/src/test/java/org/apache/sqoop/integration/server/InvalidRESTCallsTest.java, > > line 148 > > > > > > Patch ofSQOOP-2544 has removed the check of the request

Re: Review Request 37923: SQOOP-2547 Sqoop2: RESTiliency: Provide REST tests for v1/connector servlet

2015-09-08 Thread Dian Fu
> On Sept. 8, 2015, 5:39 a.m., Dian Fu wrote: > > test/src/test/java/org/apache/sqoop/integration/server/InvalidRESTCallsTest.java, > > line 148 > > > > > > Patch ofSQOOP-2544 has removed the check of the request

Re: Review Request 37923: SQOOP-2547 Sqoop2: RESTiliency: Provide REST tests for v1/connector servlet

2015-09-07 Thread Dian Fu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37923/#review97967 ---

Re: Review Request 37923: SQOOP-2547 Sqoop2: RESTiliency: Provide REST tests for v1/connector servlet

2015-09-07 Thread Dian Fu
> On Sept. 8, 2015, 5:39 a.m., Dian Fu wrote: > > Thanks Jarcec for the patch. +1 if the following issue can be resolved. - Dian --- This is an automatically generated e-mail. To reply, visit:

Review Request 37923: SQOOP-2547 Sqoop2: RESTiliency: Provide REST tests for v1/connector servlet

2015-08-29 Thread Jarek Cecho
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37923/ --- Review request for Sqoop. Bugs: SQOOP-2547