You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sqoop.apache.org by ja...@apache.org on 2015/08/27 18:11:51 UTC
sqoop git commit: SQOOP-2533: Sqoop2: Findbugs: Fix warning for
ConfigurableServlet
Repository: sqoop
Updated Branches:
refs/heads/sqoop2 a7c705938 -> e0fbca7fa
SQOOP-2533: Sqoop2: Findbugs: Fix warning for ConfigurableServlet
(Colin Ma via Jarek Jarcec Cecho)
Project: http://git-wip-us.apache.org/repos/asf/sqoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/sqoop/commit/e0fbca7f
Tree: http://git-wip-us.apache.org/repos/asf/sqoop/tree/e0fbca7f
Diff: http://git-wip-us.apache.org/repos/asf/sqoop/diff/e0fbca7f
Branch: refs/heads/sqoop2
Commit: e0fbca7faa5ee0d22294289e76700b2d59dd353b
Parents: a7c7059
Author: Jarek Jarcec Cecho <ja...@apache.org>
Authored: Thu Aug 27 09:11:27 2015 -0700
Committer: Jarek Jarcec Cecho <ja...@apache.org>
Committed: Thu Aug 27 09:11:27 2015 -0700
----------------------------------------------------------------------
.../java/org/apache/sqoop/server/v1/ConfigurableServlet.java | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/sqoop/blob/e0fbca7f/server/src/main/java/org/apache/sqoop/server/v1/ConfigurableServlet.java
----------------------------------------------------------------------
diff --git a/server/src/main/java/org/apache/sqoop/server/v1/ConfigurableServlet.java b/server/src/main/java/org/apache/sqoop/server/v1/ConfigurableServlet.java
index 4263701..7bd1901 100644
--- a/server/src/main/java/org/apache/sqoop/server/v1/ConfigurableServlet.java
+++ b/server/src/main/java/org/apache/sqoop/server/v1/ConfigurableServlet.java
@@ -37,21 +37,16 @@ import org.apache.sqoop.server.SqoopProtocolServlet;
public class ConfigurableServlet extends SqoopProtocolServlet {
private static final long serialVersionUID = 1L;
- private RequestHandler configurableRequestHandler;
private static String CONNECTOR_CONFIGURABLE = "connector";
- private static String DRIVER_CONFIGURABLE = "connector";
public ConfigurableServlet() {
- // be default
- configurableRequestHandler = new DriverRequestHandler();
}
@Override
protected JsonBean handleGetRequest(RequestContext ctx) throws Exception {
+ RequestHandler configurableRequestHandler = new DriverRequestHandler();
if (ctx.getPath().contains(CONNECTOR_CONFIGURABLE)) {
configurableRequestHandler = new ConnectorRequestHandler();
- } else if (ctx.getPath().contains(DRIVER_CONFIGURABLE)) {
- configurableRequestHandler = new DriverRequestHandler();
}
return configurableRequestHandler.handleEvent(ctx);
}