----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18401/#review35248 -----------------------------------------------------------
tools/src/main/java/org/apache/sqoop/tools/ToolRunner.java <https://reviews.apache.org/r/18401/#comment65703> minor nit. Do you think it makes sense to add a one line comment saying we don't want to kill the Tomcat container in this case? - Venkat Ranganathan On Feb. 22, 2014, 11:55 p.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18401/ > ----------------------------------------------------------- > > (Updated Feb. 22, 2014, 11:55 p.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-1290 > https://issues.apache.org/jira/browse/SQOOP-1290 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > I've provided similar solution as has been applied to Oozie via OOZIE-786. On > top of that I've also added ability to disable this behavior in case that > user will run Sqoop in Tomcat that is also running other apps. > > > Diffs > ----- > > core/src/main/java/org/apache/sqoop/core/ConfigurationConstants.java > f225fcb > server/src/main/java/org/apache/sqoop/server/ServerInitializer.java 44aea2e > tools/src/main/java/org/apache/sqoop/tools/ToolRunner.java 73dc9c5 > > Diff: https://reviews.apache.org/r/18401/diff/ > > > Testing > ------- > > Verified the fix on a real cluster. > > > Thanks, > > Jarek Cecho > >
