----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39162/#review102093 -----------------------------------------------------------
Ship it! Ship It! - Jarek Cecho On Oct. 9, 2015, 8:08 a.m., Colin Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39162/ > ----------------------------------------------------------- > > (Updated Oct. 9, 2015, 8:08 a.m.) > > > Review request for Sqoop. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > The Jetty server is instead of tomcat, the unnecessary code should be removed. > > > Diffs > ----- > > core/src/main/java/org/apache/sqoop/core/ConfigurationConstants.java > af53acb > dist/pom.xml ff18487 > dist/src/main/server/bin/setenv.bat a4d2bcc > dist/src/main/server/bin/setenv.sh 815c974 > dist/src/main/server/conf/catalina.properties 5e201be > dist/src/main/server/conf/server.xml a946da4 > dist/src/main/server/conf/sqoop.properties d85e381 > dist/src/main/server/conf/sqoop_bootstrap.properties 6e74bd3 > dist/src/main/server/webapps/ROOT/WEB-INF/web.xml 73cfa18 > dist/src/main/server/webapps/ROOT/index.html 4442532 > pom.xml e438113 > server/src/main/java/org/apache/sqoop/server/ServerInitializer.java 70bb6ee > test/pom.xml 54d54f4 > > test/src/main/java/org/apache/sqoop/test/minicluster/TomcatSqoopMiniCluster.java > a0ef78a > test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java > 5435ade > tomcat/pom.xml 3eb150b > tomcat/src/main/java/org/apache/sqoop/tomcat/TomcatToolRunner.java 75d4d10 > tools/src/main/java/org/apache/sqoop/tools/ToolRunner.java f6a84e3 > > Diff: https://reviews.apache.org/r/39162/diff/ > > > Testing > ------- > > > Thanks, > > Colin Ma > >
