----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37599/#review96584 -----------------------------------------------------------
Ship it! Ship It! - Abraham Elmahrek On Aug. 19, 2015, 1:15 a.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37599/ > ----------------------------------------------------------- > > (Updated Aug. 19, 2015, 1:15 a.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-2507 > https://issues.apache.org/jira/browse/SQOOP-2507 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > As suggested on the JIRA, I've removed code that was using default Tomcat > handlers for unsupported methods (mainly removing super calls in our parent > Servlet class). > > > Diffs > ----- > > server/src/main/java/org/apache/sqoop/handler/VersionRequestHandler.java > 2eaa0ac > server/src/main/java/org/apache/sqoop/server/SqoopProtocolServlet.java > 6b76b0b > > test/src/test/java/org/apache/sqoop/integration/server/InvalidRESTCallsTest.java > f1ef336 > > Diff: https://reviews.apache.org/r/37599/diff/ > > > Testing > ------- > > I've added integration test to verify that we're dying with expected Sqoop > specific exception. > > > Thanks, > > Jarek Cecho > >
