> On Sept. 24, 2015, 9:01 p.m., Jarek Cecho wrote: > > test/src/main/java/org/apache/sqoop/test/minicluster/JettySqoopMiniCluster.java, > > line 63 > > <https://reviews.apache.org/r/38592/diff/1-2/?file=1079643#file1079643line63> > > > > Returning empty string will just result in exception being thrown > > later. What about throwing RuntimeException here with message "server > > wasn't started" or something like that?
Good point, I'll fix this. - Colin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38592/#review100464 ----------------------------------------------------------- On Sept. 25, 2015, 7:13 a.m., Colin Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38592/ > ----------------------------------------------------------- > > (Updated Sept. 25, 2015, 7:13 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 > >
