> On Sept. 24, 2015, 9:01 p.m., Jarek Cecho wrote: > > test/src/main/java/org/apache/sqoop/test/minicluster/JettySqoopMiniCluster.java, > > line 51 > > <https://reviews.apache.org/r/38592/diff/1-2/?file=1079643#file1079643line51> > > > > If we are starting the server with "startServer" instead of > > "starServerForTest", shouldn't we also call stopServer instead of > > "stopServerForTest"?
Drop my note, I've realized that we have no "stop()" method in that class while looking at the other review. - Jarek ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38592/#review100464 ----------------------------------------------------------- On Sept. 23, 2015, 7:26 a.m., Colin Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38592/ > ----------------------------------------------------------- > > (Updated Sept. 23, 2015, 7:26 a.m.) > > > Review request for Sqoop. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Update testcases using Jetty instead of Tomcat > > > Diffs > ----- > > test/pom.xml 3e11f59 > > test/src/main/java/org/apache/sqoop/test/infrastructure/providers/SqoopInfrastructureProvider.java > 5b4f595 > > test/src/main/java/org/apache/sqoop/test/minicluster/JettySqoopMiniCluster.java > PRE-CREATION > test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java > 4452558 > test/src/main/java/org/apache/sqoop/test/testcases/JettyTestCase.java > PRE-CREATION > test/src/main/java/org/apache/sqoop/test/testcases/TomcatTestCase.java > 666749b > > test/src/test/java/org/apache/sqoop/integration/repository/derby/upgrade/DerbyRepositoryUpgradeTest.java > 1d5b692 > > Diff: https://reviews.apache.org/r/38592/diff/ > > > Testing > ------- > > > Thanks, > > Colin Ma > >
