From e0fbca7faa5ee0d22294289e76700b2d59dd353b Mon Sep 17 00:00:00 2001 From: Jarek Jarcec Cecho Date: Thu, 27 Aug 2015 09:11:27 -0700 Subject: [PATCH] SQOOP-2533: Sqoop2: Findbugs: Fix warning for ConfigurableServlet (Colin Ma via Jarek Jarcec Cecho) --- .../org/apache/sqoop/server/v1/ConfigurableServlet.java | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) 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 42637016..7bd19018 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 @@ 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); }