> On March 4, 2016, 12:29 a.m., Jarek Cecho wrote: > > server/src/main/java/org/apache/sqoop/server/SqoopProtocolServlet.java, > > line 144 > > <https://reviews.apache.org/r/43857/diff/2/?file=1268499#file1268499line144> > > > > Yup, I would take that discussion to a separate JIRA - feel free to > > create one with your thoughts :)
Ignore this comment please. - Jarek ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43857/#review121970 ----------------------------------------------------------- On Feb. 24, 2016, 11:26 p.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43857/ > ----------------------------------------------------------- > > (Updated Feb. 24, 2016, 11:26 p.m.) > > > Review request for Sqoop and Jarek Cecho. > > > Bugs: SQOOP-2545 > https://issues.apache.org/jira/browse/SQOOP-2545 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > I would like to see (&test) what happens when I call some random non-existing > REST URLs to our server. > > > Diffs > ----- > > server/src/main/java/org/apache/sqoop/handler/SubmissionRequestHandler.java > b894e37 > server/src/main/java/org/apache/sqoop/server/SqoopProtocolServlet.java > fb4a99f > > test/src/test/java/org/apache/sqoop/integration/server/rest/ConnectorRestTest.java > 839a4d0 > > test/src/test/java/org/apache/sqoop/integration/server/rest/NonExistingRestTest.java > PRE-CREATION > > test/src/test/java/org/apache/sqoop/integration/server/rest/VersionRestTest.java > e4c89df > > Diff: https://reviews.apache.org/r/43857/diff/ > > > Testing > ------- > > > Thanks, > > Jarek Cecho > >
