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 <[email protected]> Authored: Thu Aug 27 09:11:27 2015 -0700 Committer: Jarek Jarcec Cecho <[email protected]> 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); }
