Repository: ranger Updated Branches: refs/heads/master 4ce27cffb -> 0736d98e2
RANGER-1725: It is incorrect to judge whether you need to create a RangerServiceDef object for a given service Signed-off-by: peng.jianhua <peng.jian...@zte.com.cn> Project: http://git-wip-us.apache.org/repos/asf/ranger/repo Commit: http://git-wip-us.apache.org/repos/asf/ranger/commit/0736d98e Tree: http://git-wip-us.apache.org/repos/asf/ranger/tree/0736d98e Diff: http://git-wip-us.apache.org/repos/asf/ranger/diff/0736d98e Branch: refs/heads/master Commit: 0736d98e23441a112567b470254cc47bbdf2cb69 Parents: 4ce27cf Author: peng.jianhua <peng.jian...@zte.com.cn> Authored: Wed Aug 9 13:27:29 2017 +0800 Committer: peng.jianhua <peng.jian...@zte.com.cn> Committed: Thu Aug 10 03:03:07 2017 -0400 ---------------------------------------------------------------------- .../plugin/store/EmbeddedServiceDefsUtil.java | 31 +++++++++----------- 1 file changed, 14 insertions(+), 17 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ranger/blob/0736d98e/agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java ---------------------------------------------------------------------- diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java b/agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java index 0bc09f6..a09daae 100755 --- a/agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java +++ b/agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java @@ -49,7 +49,6 @@ public class EmbeddedServiceDefsUtil { // following servicedef list should be reviewed/updated whenever a new embedded service-def is added private static final String DEFAULT_BOOTSTRAP_SERVICEDEF_LIST = "tag,hdfs,hbase,hive,kms,knox,storm,yarn,kafka,solr,atlas,nifi"; private static final String PROPERTY_SUPPORTED_SERVICE_DEFS = "ranger.supportedcomponents"; - private Set<String> supportedServiceDefs; public static final String EMBEDDED_SERVICEDEF_TAG_NAME = "tag"; public static final String EMBEDDED_SERVICEDEF_HDFS_NAME = "hdfs"; public static final String EMBEDDED_SERVICEDEF_HBASE_NAME = "hbase"; @@ -81,6 +80,7 @@ public class EmbeddedServiceDefsUtil { private static EmbeddedServiceDefsUtil instance = new EmbeddedServiceDefsUtil(); private boolean createEmbeddedServiceDefs = true; + private Set<String> supportedServiceDefs; private RangerServiceDef hdfsServiceDef; private RangerServiceDef hBaseServiceDef; private RangerServiceDef hiveServiceDef; @@ -209,7 +209,7 @@ public class EmbeddedServiceDefsUtil { } RangerServiceDef ret = null; - boolean createServiceDef = (CollectionUtils.isEmpty(supportedServiceDefs) || supportedServiceDefs.contains(serviceDefName)); + boolean createServiceDef = (!CollectionUtils.isEmpty(supportedServiceDefs) && supportedServiceDefs.contains(serviceDefName)); try { ret = store.getServiceDefByName(serviceDefName); if(ret == null && createEmbeddedServiceDefs && createServiceDef) { @@ -261,23 +261,20 @@ public class EmbeddedServiceDefsUtil { return ret; } - private Set<String> getSupportedServiceDef(){ - Set<String> supportedServiceDef =new HashSet<>(); - try{ - String ranger_supportedcomponents=RangerConfiguration.getInstance().get(PROPERTY_SUPPORTED_SERVICE_DEFS, DEFAULT_BOOTSTRAP_SERVICEDEF_LIST); - if(StringUtils.isBlank(ranger_supportedcomponents) || "all".equalsIgnoreCase(ranger_supportedcomponents)){ - ranger_supportedcomponents=DEFAULT_BOOTSTRAP_SERVICEDEF_LIST; - } - String[] supportedComponents=ranger_supportedcomponents.split(","); - if(supportedComponents!=null && supportedComponents.length>0){ - for(String element:supportedComponents){ - if(!StringUtils.isBlank(element)){ - element=element.toLowerCase(); - supportedServiceDef.add(element); - } + private Set<String> getSupportedServiceDef() { + Set<String> supportedServiceDef = new HashSet<>(); + String ranger_supportedcomponents = RangerConfiguration.getInstance().get(PROPERTY_SUPPORTED_SERVICE_DEFS, DEFAULT_BOOTSTRAP_SERVICEDEF_LIST); + if (StringUtils.isBlank(ranger_supportedcomponents) || "all".equalsIgnoreCase(ranger_supportedcomponents)) { + ranger_supportedcomponents = DEFAULT_BOOTSTRAP_SERVICEDEF_LIST; + } + String[] supportedComponents = ranger_supportedcomponents.split(","); + if (supportedComponents != null && supportedComponents.length > 0) { + for (String element : supportedComponents) { + if (!StringUtils.isBlank(element)) { + element = element.toLowerCase(); + supportedServiceDef.add(element); } } - }catch(Exception ex){ } return supportedServiceDef; }