sqoop-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jar...@apache.org
Subject sqoop git commit: SQOOP-2533: Sqoop2: Findbugs: Fix warning for ConfigurableServlet
Date Thu, 27 Aug 2015 16:11:51 GMT
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 <jarcec@apache.org>
Authored: Thu Aug 27 09:11:27 2015 -0700
Committer: Jarek Jarcec Cecho <jarcec@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);
   }


Mime
View raw message