ambari-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From alejan...@apache.org
Subject ambari git commit: AMBARI-12147. When enabling ranger hbase plugin, ambari should not set hbase.rpc.protection (Velmurugan Periasamy via alejandro)
Date Fri, 26 Jun 2015 21:36:22 GMT
Repository: ambari
Updated Branches:
  refs/heads/branch-2.1 72021a94f -> 79cb0857f


AMBARI-12147. When enabling ranger hbase plugin, ambari should not set hbase.rpc.protection
(Velmurugan Periasamy via alejandro)


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/79cb0857
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/79cb0857
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/79cb0857

Branch: refs/heads/branch-2.1
Commit: 79cb0857f617ba896b29f4d5d2e665024c64d039
Parents: 72021a9
Author: Alejandro Fernandez <afernandez@hortonworks.com>
Authored: Fri Jun 26 14:36:13 2015 -0700
Committer: Alejandro Fernandez <afernandez@hortonworks.com>
Committed: Fri Jun 26 14:36:13 2015 -0700

----------------------------------------------------------------------
 .../utils/configs/modification_handlers/hbase.js    | 16 +---------------
 1 file changed, 1 insertion(+), 15 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/79cb0857/ambari-web/app/utils/configs/modification_handlers/hbase.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/utils/configs/modification_handlers/hbase.js b/ambari-web/app/utils/configs/modification_handlers/hbase.js
index 5fa1e6f..bcb87d2 100644
--- a/ambari-web/app/utils/configs/modification_handlers/hbase.js
+++ b/ambari-web/app/utils/configs/modification_handlers/hbase.js
@@ -80,7 +80,6 @@ module.exports = App.ServiceConfigModificationHandler.create({
       var configAuthEnabled = this.getConfig(allConfigs, 'hbase.security.authorization',
'hbase-site.xml', 'HBASE');
       var configMasterClasses = this.getConfig(allConfigs, 'hbase.coprocessor.master.classes',
'hbase-site.xml', 'HBASE');
       var configRegionClasses = this.getConfig(allConfigs, 'hbase.coprocessor.region.classes',
'hbase-site.xml', 'HBASE');
-      var configRpcProtection = this.getConfig(allConfigs, 'hbase.rpc.protection', 'hbase-site.xml',
'HBASE');
 
       var authEnabled = newValue == "Yes";
       var newAuthEnabledValue = authEnabled ? "true" : "false";
@@ -89,19 +88,6 @@ module.exports = App.ServiceConfigModificationHandler.create({
       // Add HBase-Ranger configs
       this.updateConfigClasses(configMasterClasses, authEnabled, affectedProperties, configAuthEnabled.get('value')
== 'true');
       this.updateConfigClasses(configRegionClasses, authEnabled, affectedProperties, configAuthEnabled.get('value')
== 'true');
-      if (newRpcProtectionValue !== configRpcProtection.get('value')) {
-        affectedProperties.push({
-          serviceName : "HBASE",
-          sourceServiceName : "HBASE",
-          propertyName : 'hbase.rpc.protection',
-          propertyDisplayName : 'hbase.rpc.protection',
-          newValue : newRpcProtectionValue,
-          curValue : configRpcProtection.get('value'),
-          changedPropertyName : hbaseAuthEnabledPropertyName,
-          removed : false,
-          filename : 'hbase-site.xml'
-        });
-      }
       if (authEnabled && newAuthEnabledValue !== configAuthEnabled.get('value'))
{
         affectedProperties.push({
           serviceName : "HBASE",
@@ -118,4 +104,4 @@ module.exports = App.ServiceConfigModificationHandler.create({
     }
     return affectedProperties;
   }
-});
\ No newline at end of file
+});


Mime
View raw message