You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by Jarek Cecho <ja...@apache.org> on 2015/08/29 15:24:56 UTC
Review Request 37922: SQOOP-2544 Sqoop2: Drop connectors servlet
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37922/
-----------------------------------------------------------
Review request for Sqoop.
Bugs: SQOOP-2544
https://issues.apache.org/jira/browse/SQOOP-2544
Repository: sqoop-sqoop2
Description
-------
Dropped the servlet as suggested on JIRA.
Diffs
-----
server/src/main/java/org/apache/sqoop/handler/ConnectorRequestHandler.java 23f6c63
server/src/main/java/org/apache/sqoop/server/v1/ConnectorsServlet.java 8184e2d
server/src/main/webapp/WEB-INF/web.xml 94cea1c
Diff: https://reviews.apache.org/r/37922/diff/
Testing
-------
Tried few integration tests and they seem to be working. Precommit hook should verify that completely.
Thanks,
Jarek Cecho
Re: Review Request 37922: SQOOP-2544 Sqoop2: Drop connectors servlet
Posted by Jarek Cecho <ja...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37922/
-----------------------------------------------------------
(Updated Sept. 24, 2015, 10:19 p.m.)
Review request for Sqoop.
Changes
-------
Rebasing.
Bugs: SQOOP-2544
https://issues.apache.org/jira/browse/SQOOP-2544
Repository: sqoop-sqoop2
Description
-------
Dropped the servlet as suggested on JIRA.
Diffs (updated)
-----
server/src/main/java/org/apache/sqoop/handler/ConnectorRequestHandler.java f9c1de2
server/src/main/java/org/apache/sqoop/server/v1/ConnectorsServlet.java 8184e2d
server/src/main/webapp/WEB-INF/web.xml 94cea1c
Diff: https://reviews.apache.org/r/37922/diff/
Testing
-------
Tried few integration tests and they seem to be working. Precommit hook should verify that completely.
Thanks,
Jarek Cecho
Re: Review Request 37922: SQOOP-2544 Sqoop2: Drop connectors servlet
Posted by Jarek Cecho <ja...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37922/
-----------------------------------------------------------
(Updated Sept. 20, 2015, 12:07 a.m.)
Review request for Sqoop.
Changes
-------
Fixed integration test added in SQOOP-2547.
Bugs: SQOOP-2544
https://issues.apache.org/jira/browse/SQOOP-2544
Repository: sqoop-sqoop2
Description
-------
Dropped the servlet as suggested on JIRA.
Diffs (updated)
-----
server/src/main/java/org/apache/sqoop/handler/ConnectorRequestHandler.java f9c1de2
server/src/main/java/org/apache/sqoop/server/v1/ConnectorsServlet.java 8184e2d
server/src/main/webapp/WEB-INF/web.xml 94cea1c
test/src/test/java/org/apache/sqoop/integration/server/InvalidRESTCallsTest.java be2ac04
Diff: https://reviews.apache.org/r/37922/diff/
Testing
-------
Tried few integration tests and they seem to be working. Precommit hook should verify that completely.
Thanks,
Jarek Cecho
Re: Review Request 37922: SQOOP-2544 Sqoop2: Drop connectors servlet
Posted by Jarek Cecho <ja...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37922/
-----------------------------------------------------------
(Updated Sept. 18, 2015, 8:12 p.m.)
Review request for Sqoop.
Changes
-------
Rebased.
Bugs: SQOOP-2544
https://issues.apache.org/jira/browse/SQOOP-2544
Repository: sqoop-sqoop2
Description
-------
Dropped the servlet as suggested on JIRA.
Diffs (updated)
-----
server/src/main/java/org/apache/sqoop/handler/ConnectorRequestHandler.java f9c1de2
server/src/main/java/org/apache/sqoop/server/v1/ConnectorsServlet.java 8184e2d
server/src/main/webapp/WEB-INF/web.xml 94cea1c
Diff: https://reviews.apache.org/r/37922/diff/
Testing
-------
Tried few integration tests and they seem to be working. Precommit hook should verify that completely.
Thanks,
Jarek Cecho