You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by Colin Ma <ju...@intel.com> on 2015/09/22 03:44:41 UTC

Review Request 38594: SQOOP-910: Sqoop2: Move to embedded jetty from tomcat

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38594/
-----------------------------------------------------------

Review request for Sqoop.


Repository: sqoop-sqoop2


Description
-------

Move to embedded jetty from tomcat


Diffs
-----

  core/src/main/java/org/apache/sqoop/audit/AuditLoggerManager.java af9282c 
  core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java b0a6841 
  core/src/main/java/org/apache/sqoop/core/SqoopServer.java 75cea65 
  core/src/main/java/org/apache/sqoop/driver/JobManager.java 923df0d 
  dist/pom.xml 37cd5ba 
  dist/src/main/bin/sqoop-sys.sh 64dd0bf 
  dist/src/main/bin/sqoop.sh 707c3fc 
  dist/src/main/conf/sqoop.properties PRE-CREATION 
  dist/src/main/conf/sqoop_bootstrap.properties PRE-CREATION 
  pom.xml 6e334a7 
  server/pom.xml 59663fa 
  server/src/main/java/org/apache/sqoop/server/SqoopJettyConstants.java PRE-CREATION 
  server/src/main/java/org/apache/sqoop/server/SqoopJettyContext.java PRE-CREATION 
  server/src/main/java/org/apache/sqoop/server/SqoopJettyServer.java PRE-CREATION 
  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 
  test/src/test/java/org/apache/sqoop/integration/server/InvalidRESTCallsTest.java be2ac04 
  tools/pom.xml 65fd40c 

Diff: https://reviews.apache.org/r/38594/diff/


Testing
-------


Thanks,

Colin Ma


Re: Review Request 38594: SQOOP-910: Sqoop2: Move to embedded jetty from tomcat

Posted by Colin Ma <ju...@intel.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38594/
-----------------------------------------------------------

(Updated Sept. 29, 2015, 8:19 a.m.)


Review request for Sqoop.


Repository: sqoop-sqoop2


Description
-------

Move to embedded jetty from tomcat


Diffs (updated)
-----

  core/src/main/java/org/apache/sqoop/audit/AuditLoggerManager.java af9282c 
  core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java b0a6841 
  core/src/main/java/org/apache/sqoop/core/SqoopServer.java 75cea65 
  core/src/main/java/org/apache/sqoop/driver/JobManager.java de1b976 
  dist/pom.xml 37cd5ba 
  dist/src/main/bin/sqoop-sys.sh 64dd0bf 
  dist/src/main/bin/sqoop.sh 707c3fc 
  dist/src/main/conf/sqoop.properties PRE-CREATION 
  dist/src/main/conf/sqoop_bootstrap.properties PRE-CREATION 
  pom.xml ef3f5f4 
  server/pom.xml 59663fa 
  server/src/main/java/org/apache/sqoop/server/SqoopJettyConstants.java PRE-CREATION 
  server/src/main/java/org/apache/sqoop/server/SqoopJettyContext.java PRE-CREATION 
  server/src/main/java/org/apache/sqoop/server/SqoopJettyServer.java PRE-CREATION 
  test/pom.xml 8218477 
  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 
  tools/pom.xml 65fd40c 

Diff: https://reviews.apache.org/r/38594/diff/


Testing
-------


Thanks,

Colin Ma


Re: Review Request 38594: SQOOP-910: Sqoop2: Move to embedded jetty from tomcat

Posted by Colin Ma <ju...@intel.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38594/
-----------------------------------------------------------

(Updated Sept. 23, 2015, 8:27 a.m.)


Review request for Sqoop.


Repository: sqoop-sqoop2


Description
-------

Move to embedded jetty from tomcat


Diffs (updated)
-----

  core/src/main/java/org/apache/sqoop/audit/AuditLoggerManager.java af9282c 
  core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java b0a6841 
  core/src/main/java/org/apache/sqoop/core/SqoopServer.java 75cea65 
  core/src/main/java/org/apache/sqoop/driver/JobManager.java de1b976 
  dist/pom.xml 37cd5ba 
  dist/src/main/bin/sqoop-sys.sh 64dd0bf 
  dist/src/main/bin/sqoop.sh 707c3fc 
  dist/src/main/conf/sqoop.properties PRE-CREATION 
  dist/src/main/conf/sqoop_bootstrap.properties PRE-CREATION 
  pom.xml 6e334a7 
  server/pom.xml 59663fa 
  server/src/main/java/org/apache/sqoop/server/SqoopJettyConstants.java PRE-CREATION 
  server/src/main/java/org/apache/sqoop/server/SqoopJettyContext.java PRE-CREATION 
  server/src/main/java/org/apache/sqoop/server/SqoopJettyServer.java PRE-CREATION 
  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 
  tools/pom.xml 65fd40c 

Diff: https://reviews.apache.org/r/38594/diff/


Testing
-------


Thanks,

Colin Ma