----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40336/#review106748 -----------------------------------------------------------
test/src/test/java/org/apache/sqoop/integration/server/InvalidRESTCallsTest.java (line 263) <https://reviews.apache.org/r/40336/#comment165505> a little concerning that this was broken and it did not impact the result of the tests. it seems that we are not actually testing the content of these responses. so why do we need to write them to a dataoutputstream? - Abraham Fine On Nov. 16, 2015, 12:15 a.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40336/ > ----------------------------------------------------------- > > (Updated Nov. 16, 2015, 12:15 a.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-2689 > https://issues.apache.org/jira/browse/SQOOP-2689 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Corrected order. > > > Diffs > ----- > > > test/src/test/java/org/apache/sqoop/integration/server/InvalidRESTCallsTest.java > d083d37 > > Diff: https://reviews.apache.org/r/40336/diff/ > > > Testing > ------- > > The integration test seems to be passing. > > > Thanks, > > Jarek Cecho > >
