You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by ab...@apache.org on 2017/03/15 00:34:34 UTC
[2/3] ranger git commit: RANGER-1426 - Change return type of
RangerBaseService.validateConfig
RANGER-1426 - Change return type of RangerBaseService.validateConfig
Signed-off-by: Colm O hEigeartaigh <co...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/ranger/commit/959ba7f0
Tree: http://git-wip-us.apache.org/repos/asf/ranger/tree/959ba7f0
Diff: http://git-wip-us.apache.org/repos/asf/ranger/diff/959ba7f0
Branch: refs/heads/ranger-0.7
Commit: 959ba7f068efdf6000e33e4e082a7b8c07f36473
Parents: 74e86ca
Author: Colm O hEigeartaigh <co...@apache.org>
Authored: Tue Mar 7 11:53:36 2017 +0000
Committer: Abhay Kulkarni <ak...@hortonworks.com>
Committed: Tue Mar 14 17:07:51 2017 -0700
----------------------------------------------------------------------
.../java/org/apache/ranger/plugin/client/BaseClient.java | 3 +--
.../org/apache/ranger/plugin/service/RangerBaseService.java | 3 +--
.../org/apache/ranger/services/tag/RangerServiceTag.java | 2 +-
.../org/apache/ranger/services/hbase/RangerServiceHBase.java | 4 ++--
.../org/apache/ranger/services/hbase/client/HBaseClient.java | 4 ++--
.../ranger/services/hbase/client/HBaseResourceMgr.java | 5 ++---
.../org/apache/ranger/services/hdfs/RangerServiceHdfs.java | 4 ++--
.../org/apache/ranger/services/hdfs/client/HdfsClient.java | 4 ++--
.../apache/ranger/services/hdfs/client/HdfsResourceMgr.java | 5 ++---
.../org/apache/ranger/services/hive/RangerServiceHive.java | 4 ++--
.../org/apache/ranger/services/hive/client/HiveClient.java | 4 ++--
.../apache/ranger/services/hive/client/HiveResourceMgr.java | 5 ++---
.../org/apache/ranger/services/knox/RangerServiceKnox.java | 4 ++--
.../org/apache/ranger/services/knox/client/KnoxClient.java | 4 ++--
.../apache/ranger/services/knox/client/KnoxResourceMgr.java | 5 ++---
.../org/apache/ranger/services/atlas/RangerServiceAtlas.java | 4 ++--
.../ranger/services/atlas/client/AtlasResourceMgr.java | 5 ++---
.../org/apache/ranger/services/kafka/RangerServiceKafka.java | 6 ++++--
.../ranger/services/kafka/client/ServiceKafkaClient.java | 4 ++--
.../services/kafka/client/ServiceKafkaConnectionMgr.java | 3 +--
.../org/apache/ranger/services/kms/RangerServiceKMS.java | 4 ++--
.../org/apache/ranger/services/kms/client/KMSClient.java | 4 ++--
.../apache/ranger/services/kms/client/KMSResourceMgr.java | 5 ++---
.../org/apache/ranger/services/solr/RangerServiceSolr.java | 6 ++++--
.../ranger/services/solr/client/ServiceSolrClient.java | 4 ++--
.../services/solr/client/ServiceSolrConnectionMgr.java | 3 +--
.../org/apache/ranger/services/yarn/RangerServiceYarn.java | 4 ++--
.../org/apache/ranger/services/yarn/client/YarnClient.java | 4 ++--
.../apache/ranger/services/yarn/client/YarnResourceMgr.java | 5 ++---
.../src/main/java/org/apache/ranger/biz/ServiceMgr.java | 8 ++++----
.../org/apache/ranger/services/storm/RangerServiceStorm.java | 4 ++--
.../org/apache/ranger/services/storm/client/StormClient.java | 4 ++--
.../ranger/services/storm/client/StormResourceMgr.java | 5 ++---
33 files changed, 67 insertions(+), 75 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/agents-common/src/main/java/org/apache/ranger/plugin/client/BaseClient.java
----------------------------------------------------------------------
diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/client/BaseClient.java b/agents-common/src/main/java/org/apache/ranger/plugin/client/BaseClient.java
index aa90a92..70682a6 100644
--- a/agents-common/src/main/java/org/apache/ranger/plugin/client/BaseClient.java
+++ b/agents-common/src/main/java/org/apache/ranger/plugin/client/BaseClient.java
@@ -21,7 +21,6 @@
import java.io.IOException;
import java.util.ArrayList;
-import java.util.HashMap;
import java.util.List;
import java.util.Map;
@@ -163,7 +162,7 @@ public abstract class BaseClient {
public static void generateResponseDataMap(boolean connectivityStatus,
String message, String description, Long objectId,
- String fieldName, HashMap<String, Object> responseData) {
+ String fieldName, Map<String, Object> responseData) {
responseData.put("connectivityStatus", connectivityStatus);
responseData.put("message", message);
responseData.put("description", description);
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBaseService.java
----------------------------------------------------------------------
diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBaseService.java b/agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBaseService.java
index 73027a0..debaa83 100644
--- a/agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBaseService.java
+++ b/agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBaseService.java
@@ -19,7 +19,6 @@
package org.apache.ranger.plugin.service;
-import java.util.HashMap;
import java.util.List;
import java.util.Map;
@@ -82,7 +81,7 @@ public abstract class RangerBaseService {
this.serviceType = serviceType;
}
- public abstract HashMap<String, Object> validateConfig() throws Exception;
+ public abstract Map<String, Object> validateConfig() throws Exception;
public abstract List<String> lookupResource(ResourceLookupContext context) throws Exception;
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/agents-common/src/main/java/org/apache/ranger/services/tag/RangerServiceTag.java
----------------------------------------------------------------------
diff --git a/agents-common/src/main/java/org/apache/ranger/services/tag/RangerServiceTag.java b/agents-common/src/main/java/org/apache/ranger/services/tag/RangerServiceTag.java
index 3940154..d3085d4 100644
--- a/agents-common/src/main/java/org/apache/ranger/services/tag/RangerServiceTag.java
+++ b/agents-common/src/main/java/org/apache/ranger/services/tag/RangerServiceTag.java
@@ -56,7 +56,7 @@ public class RangerServiceTag extends RangerBaseService {
}
@Override
- public HashMap<String,Object> validateConfig() throws Exception {
+ public Map<String,Object> validateConfig() throws Exception {
if(LOG.isDebugEnabled()) {
LOG.debug("==> RangerServiceTag.validateConfig(" + serviceName + " )");
}
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/hbase-agent/src/main/java/org/apache/ranger/services/hbase/RangerServiceHBase.java
----------------------------------------------------------------------
diff --git a/hbase-agent/src/main/java/org/apache/ranger/services/hbase/RangerServiceHBase.java b/hbase-agent/src/main/java/org/apache/ranger/services/hbase/RangerServiceHBase.java
index 7d5bf9b..01b97ea 100644
--- a/hbase-agent/src/main/java/org/apache/ranger/services/hbase/RangerServiceHBase.java
+++ b/hbase-agent/src/main/java/org/apache/ranger/services/hbase/RangerServiceHBase.java
@@ -46,8 +46,8 @@ public class RangerServiceHBase extends RangerBaseService {
}
@Override
- public HashMap<String,Object> validateConfig() throws Exception {
- HashMap<String, Object> ret = new HashMap<String, Object>();
+ public Map<String,Object> validateConfig() throws Exception {
+ Map<String, Object> ret = new HashMap<String, Object>();
String serviceName = getServiceName();
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseClient.java
----------------------------------------------------------------------
diff --git a/hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseClient.java b/hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseClient.java
index f0f086c..d9870e3 100644
--- a/hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseClient.java
+++ b/hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseClient.java
@@ -85,10 +85,10 @@ public class HBaseClient extends BaseClient {
return connectionProp;
}
- public static HashMap<String, Object> connectionTest (String dataSource,
+ public static Map<String, Object> connectionTest (String dataSource,
Map<String, String> configs) throws Exception {
- HashMap<String, Object> responseData = new HashMap<String, Object>();
+ Map<String, Object> responseData = new HashMap<String, Object>();
final String errMsg = " You can still save the repository and start creating "
+ "policies, but you would not be able to use autocomplete for "
+ "resource names. Check ranger_admin.log for more info.";
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseResourceMgr.java
----------------------------------------------------------------------
diff --git a/hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseResourceMgr.java b/hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseResourceMgr.java
index c033b00..5db6277 100644
--- a/hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseResourceMgr.java
+++ b/hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseResourceMgr.java
@@ -19,7 +19,6 @@
package org.apache.ranger.services.hbase.client;
-import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.concurrent.Callable;
@@ -38,8 +37,8 @@ public class HBaseResourceMgr {
private static final String TABLE = "table";
private static final String COLUMNFAMILY = "column-family";
- public static HashMap<String, Object> connectionTest(String serviceName, Map<String, String> configs) throws Exception {
- HashMap<String, Object> ret = null;
+ public static Map<String, Object> connectionTest(String serviceName, Map<String, String> configs) throws Exception {
+ Map<String, Object> ret = null;
if(LOG.isDebugEnabled()) {
LOG.debug("<== HBaseResourceMgr.connectionTest() ServiceName: "+ serviceName + "Configs" + configs );
}
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/RangerServiceHdfs.java
----------------------------------------------------------------------
diff --git a/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/RangerServiceHdfs.java b/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/RangerServiceHdfs.java
index e1648dc..bc12da9 100644
--- a/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/RangerServiceHdfs.java
+++ b/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/RangerServiceHdfs.java
@@ -46,8 +46,8 @@ public class RangerServiceHdfs extends RangerBaseService {
}
@Override
- public HashMap<String,Object> validateConfig() throws Exception {
- HashMap<String, Object> ret = new HashMap<String, Object>();
+ public Map<String,Object> validateConfig() throws Exception {
+ Map<String, Object> ret = new HashMap<String, Object>();
String serviceName = getServiceName();
if(LOG.isDebugEnabled()) {
LOG.debug("<== RangerServiceHdfs.validateConfig Service: (" + serviceName + " )");
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsClient.java
----------------------------------------------------------------------
diff --git a/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsClient.java b/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsClient.java
index d712b08..4db0ec1 100644
--- a/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsClient.java
+++ b/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsClient.java
@@ -198,11 +198,11 @@ public class HdfsClient extends BaseClient {
}
}
- public static HashMap<String, Object> connectionTest(String serviceName,
+ public static Map<String, Object> connectionTest(String serviceName,
Map<String, String> configs) throws Exception {
LOG.info("===> HdfsClient.testConnection()" );
- HashMap<String, Object> responseData = new HashMap<String, Object>();
+ Map<String, Object> responseData = new HashMap<String, Object>();
boolean connectivityStatus = false;
String validateConfigsMsg = null;
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsResourceMgr.java
----------------------------------------------------------------------
diff --git a/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsResourceMgr.java b/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsResourceMgr.java
index bb6aa496..ffbab2a 100644
--- a/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsResourceMgr.java
+++ b/hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsResourceMgr.java
@@ -20,7 +20,6 @@
package org.apache.ranger.services.hdfs.client;
import java.util.ArrayList;
-import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.concurrent.Callable;
@@ -36,8 +35,8 @@ public class HdfsResourceMgr {
private static final Logger LOG = Logger.getLogger(HdfsResourceMgr.class);
public static final String PATH = "path";
- public static HashMap<String, Object> connectionTest(String serviceName, Map<String, String> configs) throws Exception {
- HashMap<String, Object> ret = null;
+ public static Map<String, Object> connectionTest(String serviceName, Map<String, String> configs) throws Exception {
+ Map<String, Object> ret = null;
if(LOG.isDebugEnabled()) {
LOG.debug("<== HdfsResourceMgr.connectionTest ServiceName: "+ serviceName + "Configs" + configs );
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/hive-agent/src/main/java/org/apache/ranger/services/hive/RangerServiceHive.java
----------------------------------------------------------------------
diff --git a/hive-agent/src/main/java/org/apache/ranger/services/hive/RangerServiceHive.java b/hive-agent/src/main/java/org/apache/ranger/services/hive/RangerServiceHive.java
index 9ce0347..a2bd5f7 100644
--- a/hive-agent/src/main/java/org/apache/ranger/services/hive/RangerServiceHive.java
+++ b/hive-agent/src/main/java/org/apache/ranger/services/hive/RangerServiceHive.java
@@ -46,8 +46,8 @@ public class RangerServiceHive extends RangerBaseService {
}
@Override
- public HashMap<String,Object> validateConfig() throws Exception {
- HashMap<String, Object> ret = new HashMap<String, Object>();
+ public Map<String,Object> validateConfig() throws Exception {
+ Map<String, Object> ret = new HashMap<String, Object>();
String serviceName = getServiceName();
if(LOG.isDebugEnabled()) {
LOG.debug("==> RangerServiceHive.validateConfig Service: (" + serviceName + " )");
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveClient.java
----------------------------------------------------------------------
diff --git a/hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveClient.java b/hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveClient.java
index ec61458..734c8e7 100644
--- a/hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveClient.java
+++ b/hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveClient.java
@@ -620,10 +620,10 @@ public class HiveClient extends BaseClient implements Closeable {
}
}
- public static HashMap<String, Object> connectionTest(String serviceName,
+ public static Map<String, Object> connectionTest(String serviceName,
Map<String, String> connectionProperties) throws Exception {
HiveClient connectionObj = null;
- HashMap<String, Object> responseData = new HashMap<String, Object>();
+ Map<String, Object> responseData = new HashMap<String, Object>();
boolean connectivityStatus = false;
String errMsg = " You can still save the repository and start creating "
+ "policies, but you would not be able to use autocomplete for "
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveResourceMgr.java
----------------------------------------------------------------------
diff --git a/hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveResourceMgr.java b/hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveResourceMgr.java
index 0a15f70..f1da958 100644
--- a/hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveResourceMgr.java
+++ b/hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveResourceMgr.java
@@ -18,7 +18,6 @@
*/
package org.apache.ranger.services.hive.client;
-import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.concurrent.Callable;
@@ -38,8 +37,8 @@ public class HiveResourceMgr {
private static final String COLUMN = "column";
- public static HashMap<String, Object> connectionTest(String serviceName, Map<String, String> configs) throws Exception {
- HashMap<String, Object> ret = null;
+ public static Map<String, Object> connectionTest(String serviceName, Map<String, String> configs) throws Exception {
+ Map<String, Object> ret = null;
if(LOG.isDebugEnabled()) {
LOG.debug("==> HiveResourceMgr.connectionTest ServiceName: "+ serviceName + "Configs" + configs );
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/knox-agent/src/main/java/org/apache/ranger/services/knox/RangerServiceKnox.java
----------------------------------------------------------------------
diff --git a/knox-agent/src/main/java/org/apache/ranger/services/knox/RangerServiceKnox.java b/knox-agent/src/main/java/org/apache/ranger/services/knox/RangerServiceKnox.java
index 813b690..b72e776 100644
--- a/knox-agent/src/main/java/org/apache/ranger/services/knox/RangerServiceKnox.java
+++ b/knox-agent/src/main/java/org/apache/ranger/services/knox/RangerServiceKnox.java
@@ -45,8 +45,8 @@ public class RangerServiceKnox extends RangerBaseService {
}
@Override
- public HashMap<String,Object> validateConfig() throws Exception {
- HashMap<String, Object> ret = new HashMap<String, Object>();
+ public Map<String,Object> validateConfig() throws Exception {
+ Map<String, Object> ret = new HashMap<String, Object>();
String serviceName = getServiceName();
if(LOG.isDebugEnabled()) {
LOG.debug("==> RangerServiceKnox.validateConfig Service: (" + serviceName + " )");
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxClient.java
----------------------------------------------------------------------
diff --git a/knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxClient.java b/knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxClient.java
index 4c389e3..ec08ae8 100644
--- a/knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxClient.java
+++ b/knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxClient.java
@@ -302,14 +302,14 @@ public class KnoxClient {
}
}
- public static HashMap<String, Object> connectionTest(String serviceName,
+ public static Map<String, Object> connectionTest(String serviceName,
Map<String, String> configs) {
String errMsg = " You can still save the repository and start creating "
+ "policies, but you would not be able to use autocomplete for "
+ "resource names. Check ranger_admin.log for more info.";
boolean connectivityStatus = false;
- HashMap<String, Object> responseData = new HashMap<String, Object>();
+ Map<String, Object> responseData = new HashMap<String, Object>();
KnoxClient knoxClient = getKnoxClient(serviceName, configs);
List<String> strList = getKnoxResources(knoxClient, "", null,null,null);
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxResourceMgr.java
----------------------------------------------------------------------
diff --git a/knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxResourceMgr.java b/knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxResourceMgr.java
index 1d086fb..e887b11 100644
--- a/knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxResourceMgr.java
+++ b/knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxResourceMgr.java
@@ -19,7 +19,6 @@
package org.apache.ranger.services.knox.client;
-import java.util.HashMap;
import java.util.List;
import java.util.Map;
@@ -34,8 +33,8 @@ public class KnoxResourceMgr {
private static final String TOPOLOGY = "topology";
private static final String SERVICE = "service";
- public static HashMap<String, Object> validateConfig(String serviceName, Map<String, String> configs) throws Exception {
- HashMap<String, Object> ret = null;
+ public static Map<String, Object> validateConfig(String serviceName, Map<String, String> configs) throws Exception {
+ Map<String, Object> ret = null;
if (LOG.isDebugEnabled()) {
LOG.debug("==> KnoxResourceMgr.testConnection ServiceName: "+ serviceName + "Configs" + configs );
}
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-atlas/src/main/java/org/apache/ranger/services/atlas/RangerServiceAtlas.java
----------------------------------------------------------------------
diff --git a/plugin-atlas/src/main/java/org/apache/ranger/services/atlas/RangerServiceAtlas.java b/plugin-atlas/src/main/java/org/apache/ranger/services/atlas/RangerServiceAtlas.java
index 7e7dfe3..d2b60bd 100644
--- a/plugin-atlas/src/main/java/org/apache/ranger/services/atlas/RangerServiceAtlas.java
+++ b/plugin-atlas/src/main/java/org/apache/ranger/services/atlas/RangerServiceAtlas.java
@@ -44,8 +44,8 @@ public class RangerServiceAtlas extends RangerBaseService {
}
@Override
- public HashMap<String, Object> validateConfig() throws Exception {
- HashMap<String, Object> responseMap = new HashMap<String, Object>();
+ public Map<String, Object> validateConfig() throws Exception {
+ Map<String, Object> responseMap = new HashMap<String, Object>();
String serviceName = getServiceName();
if (LOG.isDebugEnabled()) {
LOG.debug("==> RangerServiceAtlas.validateConfig Service: ("+ serviceName + " )");
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasResourceMgr.java
----------------------------------------------------------------------
diff --git a/plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasResourceMgr.java b/plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasResourceMgr.java
index cb2ac9a..f81e304 100644
--- a/plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasResourceMgr.java
+++ b/plugin-atlas/src/main/java/org/apache/ranger/services/atlas/client/AtlasResourceMgr.java
@@ -19,7 +19,6 @@
package org.apache.ranger.services.atlas.client;
-import java.util.HashMap;
import java.util.List;
import java.util.Map;
@@ -29,9 +28,9 @@ import org.apache.ranger.plugin.service.ResourceLookupContext;
public class AtlasResourceMgr {
private static final Logger LOG = Logger.getLogger(AtlasResourceMgr.class);
- public static HashMap<String, Object> validateConfig(String serviceName, Map<String, String> configs) throws Exception {
+ public static Map<String, Object> validateConfig(String serviceName, Map<String, String> configs) throws Exception {
- HashMap<String, Object> ret = null;
+ Map<String, Object> ret = null;
if(LOG.isDebugEnabled()) {
LOG.debug("==> AtlasResourceMgr.validateConfig ServiceName: "+ serviceName + "Configs" + configs );
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/RangerServiceKafka.java
----------------------------------------------------------------------
diff --git a/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/RangerServiceKafka.java b/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/RangerServiceKafka.java
index 2f031fa..86e97bc 100644
--- a/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/RangerServiceKafka.java
+++ b/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/RangerServiceKafka.java
@@ -21,6 +21,8 @@ package org.apache.ranger.services.kafka;
import java.util.HashMap;
import java.util.List;
+import java.util.Map;
+
import org.apache.ranger.plugin.model.RangerService;
import org.apache.ranger.plugin.model.RangerServiceDef;
import org.apache.ranger.plugin.service.RangerBaseService;
@@ -43,8 +45,8 @@ public class RangerServiceKafka extends RangerBaseService {
}
@Override
- public HashMap<String, Object> validateConfig() throws Exception {
- HashMap<String, Object> ret = new HashMap<String, Object>();
+ public Map<String, Object> validateConfig() throws Exception {
+ Map<String, Object> ret = new HashMap<String, Object>();
if (LOG.isDebugEnabled()) {
LOG.debug("==> RangerServiceKafka.validateConfig(" + serviceName + ")");
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/client/ServiceKafkaClient.java
----------------------------------------------------------------------
diff --git a/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/client/ServiceKafkaClient.java b/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/client/ServiceKafkaClient.java
index c908550..6929257 100644
--- a/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/client/ServiceKafkaClient.java
+++ b/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/client/ServiceKafkaClient.java
@@ -59,9 +59,9 @@ public class ServiceKafkaClient {
this.zookeeperConnect = zookeeperConnect;
}
- public HashMap<String, Object> connectionTest() throws Exception {
+ public Map<String, Object> connectionTest() throws Exception {
String errMsg = errMessage;
- HashMap<String, Object> responseData = new HashMap<String, Object>();
+ Map<String, Object> responseData = new HashMap<String, Object>();
try {
getTopicList(null);
// If it doesn't throw exception, then assume the instance is
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/client/ServiceKafkaConnectionMgr.java
----------------------------------------------------------------------
diff --git a/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/client/ServiceKafkaConnectionMgr.java b/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/client/ServiceKafkaConnectionMgr.java
index 0056c84..9e0d6b4 100644
--- a/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/client/ServiceKafkaConnectionMgr.java
+++ b/plugin-kafka/src/main/java/org/apache/ranger/services/kafka/client/ServiceKafkaConnectionMgr.java
@@ -19,7 +19,6 @@
package org.apache.ranger.services.kafka.client;
-import java.util.HashMap;
import java.util.Map;
public class ServiceKafkaConnectionMgr {
@@ -41,7 +40,7 @@ public class ServiceKafkaConnectionMgr {
* @param configs
* @return
*/
- public static HashMap<String, Object> connectionTest(String serviceName,
+ public static Map<String, Object> connectionTest(String serviceName,
Map<String, String> configs) throws Exception {
ServiceKafkaClient serviceKafkaClient = getKafkaClient(serviceName,
configs);
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-kms/src/main/java/org/apache/ranger/services/kms/RangerServiceKMS.java
----------------------------------------------------------------------
diff --git a/plugin-kms/src/main/java/org/apache/ranger/services/kms/RangerServiceKMS.java b/plugin-kms/src/main/java/org/apache/ranger/services/kms/RangerServiceKMS.java
index 8b41ae3..7657099 100644
--- a/plugin-kms/src/main/java/org/apache/ranger/services/kms/RangerServiceKMS.java
+++ b/plugin-kms/src/main/java/org/apache/ranger/services/kms/RangerServiceKMS.java
@@ -44,8 +44,8 @@ public class RangerServiceKMS extends RangerBaseService {
}
@Override
- public HashMap<String,Object> validateConfig() throws Exception {
- HashMap<String, Object> ret = new HashMap<String, Object>();
+ public Map<String,Object> validateConfig() throws Exception {
+ Map<String, Object> ret = new HashMap<String, Object>();
String serviceName = getServiceName();
if(LOG.isDebugEnabled()) {
LOG.debug("==> RangerServiceKMS.validateConfig Service: (" + serviceName + " )");
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSClient.java
----------------------------------------------------------------------
diff --git a/plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSClient.java b/plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSClient.java
index 016065d..af0ac71 100755
--- a/plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSClient.java
+++ b/plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSClient.java
@@ -305,13 +305,13 @@ public class KMSClient {
return lret;
}
- public static HashMap<String, Object> testConnection(String serviceName,
+ public static Map<String, Object> testConnection(String serviceName,
Map<String, String> configs) {
List<String> strList = new ArrayList<String>();
String errMsg = errMessage;
boolean connectivityStatus = false;
- HashMap<String, Object> responseData = new HashMap<String, Object>();
+ Map<String, Object> responseData = new HashMap<String, Object>();
KMSClient kmsClient = getKmsClient(serviceName, configs);
strList = getKmsKey(kmsClient, "", null);
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSResourceMgr.java
----------------------------------------------------------------------
diff --git a/plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSResourceMgr.java b/plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSResourceMgr.java
index 09f2a6f..fe54723 100755
--- a/plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSResourceMgr.java
+++ b/plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSResourceMgr.java
@@ -19,7 +19,6 @@
package org.apache.ranger.services.kms.client;
-import java.util.HashMap;
import java.util.List;
import java.util.Map;
@@ -30,8 +29,8 @@ public class KMSResourceMgr {
private static final Logger LOG = Logger.getLogger(KMSResourceMgr.class);
private static final String KMSKEY = "keyname";
- public static HashMap<String, Object> validateConfig(String serviceName, Map<String, String> configs) throws Exception {
- HashMap<String, Object> ret = null;
+ public static Map<String, Object> validateConfig(String serviceName, Map<String, String> configs) throws Exception {
+ Map<String, Object> ret = null;
if(LOG.isDebugEnabled()) {
LOG.debug("==> KMSResourceMgr.validateConfig ServiceName: "+ serviceName + "Configs" + configs );
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-solr/src/main/java/org/apache/ranger/services/solr/RangerServiceSolr.java
----------------------------------------------------------------------
diff --git a/plugin-solr/src/main/java/org/apache/ranger/services/solr/RangerServiceSolr.java b/plugin-solr/src/main/java/org/apache/ranger/services/solr/RangerServiceSolr.java
index 8fa51ea..6477af7 100644
--- a/plugin-solr/src/main/java/org/apache/ranger/services/solr/RangerServiceSolr.java
+++ b/plugin-solr/src/main/java/org/apache/ranger/services/solr/RangerServiceSolr.java
@@ -21,6 +21,8 @@ package org.apache.ranger.services.solr;
import java.util.HashMap;
import java.util.List;
+import java.util.Map;
+
import org.apache.ranger.plugin.model.RangerService;
import org.apache.ranger.plugin.model.RangerServiceDef;
import org.apache.ranger.plugin.service.RangerBaseService;
@@ -44,8 +46,8 @@ public class RangerServiceSolr extends RangerBaseService {
}
@Override
- public HashMap<String, Object> validateConfig() throws Exception {
- HashMap<String, Object> ret = new HashMap<String, Object>();
+ public Map<String, Object> validateConfig() throws Exception {
+ Map<String, Object> ret = new HashMap<String, Object>();
String serviceName = getServiceName();
if (LOG.isDebugEnabled()) {
LOG.debug("==> RangerServiceSolr.validateConfig Service: ("
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-solr/src/main/java/org/apache/ranger/services/solr/client/ServiceSolrClient.java
----------------------------------------------------------------------
diff --git a/plugin-solr/src/main/java/org/apache/ranger/services/solr/client/ServiceSolrClient.java b/plugin-solr/src/main/java/org/apache/ranger/services/solr/client/ServiceSolrClient.java
index 74da115..bc5fdd0 100644
--- a/plugin-solr/src/main/java/org/apache/ranger/services/solr/client/ServiceSolrClient.java
+++ b/plugin-solr/src/main/java/org/apache/ranger/services/solr/client/ServiceSolrClient.java
@@ -71,9 +71,9 @@ public class ServiceSolrClient {
}
- public HashMap<String, Object> connectionTest() throws Exception {
+ public Map<String, Object> connectionTest() throws Exception {
String errMsg = errMessage;
- HashMap<String, Object> responseData = new HashMap<String, Object>();
+ Map<String, Object> responseData = new HashMap<String, Object>();
try {
getCollectionList(null);
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-solr/src/main/java/org/apache/ranger/services/solr/client/ServiceSolrConnectionMgr.java
----------------------------------------------------------------------
diff --git a/plugin-solr/src/main/java/org/apache/ranger/services/solr/client/ServiceSolrConnectionMgr.java b/plugin-solr/src/main/java/org/apache/ranger/services/solr/client/ServiceSolrConnectionMgr.java
index 81b664c..d8470a0 100644
--- a/plugin-solr/src/main/java/org/apache/ranger/services/solr/client/ServiceSolrConnectionMgr.java
+++ b/plugin-solr/src/main/java/org/apache/ranger/services/solr/client/ServiceSolrConnectionMgr.java
@@ -19,7 +19,6 @@
package org.apache.ranger.services.solr.client;
-import java.util.HashMap;
import java.util.Map;
import org.apache.solr.client.solrj.SolrClient;
@@ -49,7 +48,7 @@ public class ServiceSolrConnectionMgr {
* @param configs
* @return
*/
- public static HashMap<String, Object> connectionTest(String serviceName,
+ public static Map<String, Object> connectionTest(String serviceName,
Map<String, String> configs) throws Exception {
ServiceSolrClient serviceSolrClient = getSolrClient(serviceName,
configs);
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/RangerServiceYarn.java
----------------------------------------------------------------------
diff --git a/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/RangerServiceYarn.java b/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/RangerServiceYarn.java
index 7b11ded..69f2bc3 100644
--- a/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/RangerServiceYarn.java
+++ b/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/RangerServiceYarn.java
@@ -44,8 +44,8 @@ public class RangerServiceYarn extends RangerBaseService {
}
@Override
- public HashMap<String,Object> validateConfig() throws Exception {
- HashMap<String, Object> ret = new HashMap<String, Object>();
+ public Map<String,Object> validateConfig() throws Exception {
+ Map<String, Object> ret = new HashMap<String, Object>();
String serviceName = getServiceName();
if(LOG.isDebugEnabled()) {
LOG.debug("==> RangerServiceYarn.validateConfig Service: (" + serviceName + " )");
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java
----------------------------------------------------------------------
diff --git a/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java b/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java
index e8ad7b9..9188bd7 100644
--- a/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java
+++ b/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java
@@ -249,12 +249,12 @@ public class YarnClient extends BaseClient {
return ret;
}
- public static HashMap<String, Object> connectionTest(String serviceName,
+ public static Map<String, Object> connectionTest(String serviceName,
Map<String, String> configs) {
String errMsg = errMessage;
boolean connectivityStatus = false;
- HashMap<String, Object> responseData = new HashMap<String, Object>();
+ Map<String, Object> responseData = new HashMap<String, Object>();
YarnClient yarnClient = getYarnClient(serviceName,
configs);
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnResourceMgr.java
----------------------------------------------------------------------
diff --git a/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnResourceMgr.java b/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnResourceMgr.java
index 35b138f..991db07 100644
--- a/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnResourceMgr.java
+++ b/plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnResourceMgr.java
@@ -19,7 +19,6 @@
package org.apache.ranger.services.yarn.client;
-import java.util.HashMap;
import java.util.List;
import java.util.Map;
@@ -30,8 +29,8 @@ public class YarnResourceMgr {
private static final Logger LOG = Logger.getLogger(YarnResourceMgr.class);
private static final String YARNQUEUE = "queue";
- public static HashMap<String, Object> validateConfig(String serviceName, Map<String, String> configs) throws Exception {
- HashMap<String, Object> ret = null;
+ public static Map<String, Object> validateConfig(String serviceName, Map<String, String> configs) throws Exception {
+ Map<String, Object> ret = null;
if(LOG.isDebugEnabled()) {
LOG.debug("==> YarnResourceMgr.validateConfig ServiceName: "+ serviceName + "Configs" + configs );
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java
----------------------------------------------------------------------
diff --git a/security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java b/security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java
index f512049..f2b8f99 100644
--- a/security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java
+++ b/security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java
@@ -174,7 +174,7 @@ public class ServiceMgr {
// Timeout value use during validate config is 10 times that used during lookup
long time = getTimeoutValueForValidateConfigInMilliSeconds(svc);
ValidateCallable callable = new ValidateCallable(svc);
- HashMap<String, Object> responseData = timedExecutor.timedTask(callable, time, TimeUnit.MILLISECONDS);
+ Map<String, Object> responseData = timedExecutor.timedTask(callable, time, TimeUnit.MILLISECONDS);
ret = generateResponseForTestConn(responseData, "");
} catch (Exception e) {
@@ -373,7 +373,7 @@ public class ServiceMgr {
}
private VXResponse generateResponseForTestConn(
- HashMap<String, Object> responseData, String msg) {
+ Map<String, Object> responseData, String msg) {
VXResponse vXResponse = new VXResponse();
Long objId = null;
@@ -544,7 +544,7 @@ public class ServiceMgr {
}
}
- static class ValidateCallable extends TimedCallable<HashMap<String, Object>> {
+ static class ValidateCallable extends TimedCallable<Map<String, Object>> {
public ValidateCallable(RangerBaseService svc) {
super(svc);
@@ -556,7 +556,7 @@ public class ServiceMgr {
}
@Override
- public HashMap<String, Object> actualCall() throws Exception {
+ public Map<String, Object> actualCall() throws Exception {
return svc.validateConfig();
}
}
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/storm-agent/src/main/java/org/apache/ranger/services/storm/RangerServiceStorm.java
----------------------------------------------------------------------
diff --git a/storm-agent/src/main/java/org/apache/ranger/services/storm/RangerServiceStorm.java b/storm-agent/src/main/java/org/apache/ranger/services/storm/RangerServiceStorm.java
index dccd311..d4dbd7b 100644
--- a/storm-agent/src/main/java/org/apache/ranger/services/storm/RangerServiceStorm.java
+++ b/storm-agent/src/main/java/org/apache/ranger/services/storm/RangerServiceStorm.java
@@ -45,8 +45,8 @@ public class RangerServiceStorm extends RangerBaseService {
}
@Override
- public HashMap<String,Object> validateConfig() throws Exception {
- HashMap<String, Object> ret = new HashMap<String, Object>();
+ public Map<String,Object> validateConfig() throws Exception {
+ Map<String, Object> ret = new HashMap<String, Object>();
String serviceName = getServiceName();
if(LOG.isDebugEnabled()) {
LOG.debug("==> RangerServiceStorm.validateConfig Service: (" + serviceName + " )");
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormClient.java
----------------------------------------------------------------------
diff --git a/storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormClient.java b/storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormClient.java
index cb56609..b72a9a2 100644
--- a/storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormClient.java
+++ b/storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormClient.java
@@ -348,12 +348,12 @@ public class StormClient {
return ret;
}
- public static HashMap<String, Object> connectionTest(String serviceName,
+ public static Map<String, Object> connectionTest(String serviceName,
Map<String, String> configs) {
String errMsg = errMessage;
boolean connectivityStatus = false;
- HashMap<String, Object> responseData = new HashMap<String, Object>();
+ Map<String, Object> responseData = new HashMap<String, Object>();
StormClient stormClient = getStormClient(serviceName,
configs);
http://git-wip-us.apache.org/repos/asf/ranger/blob/959ba7f0/storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormResourceMgr.java
----------------------------------------------------------------------
diff --git a/storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormResourceMgr.java b/storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormResourceMgr.java
index 43960b3..0dd5507 100644
--- a/storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormResourceMgr.java
+++ b/storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormResourceMgr.java
@@ -20,7 +20,6 @@
package org.apache.ranger.services.storm.client;
import java.util.ArrayList;
-import java.util.HashMap;
import java.util.List;
import java.util.Map;
@@ -31,8 +30,8 @@ public class StormResourceMgr {
private static final Logger LOG = Logger.getLogger(StormResourceMgr.class);
private static final String TOPOLOGY = "topology";
- public static HashMap<String, Object> validateConfig(String serviceName, Map<String, String> configs) throws Exception {
- HashMap<String, Object> ret = null;
+ public static Map<String, Object> validateConfig(String serviceName, Map<String, String> configs) throws Exception {
+ Map<String, Object> ret = null;
if(LOG.isDebugEnabled()) {
LOG.debug("==> StormResourceMgr.validateConfig ServiceName: "+ serviceName + "Configs" + configs );