You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by al...@apache.org on 2015/06/26 23:34:59 UTC

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

Repository: ambari
Updated Branches:
  refs/heads/trunk 6d889f0cb -> 4a0c4098c


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/4a0c4098
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/4a0c4098
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/4a0c4098

Branch: refs/heads/trunk
Commit: 4a0c4098c6476bcb6ecdc0138a619f3277c31db9
Parents: 6d889f0
Author: Alejandro Fernandez <af...@hortonworks.com>
Authored: Fri Jun 26 14:34:52 2015 -0700
Committer: Alejandro Fernandez <af...@hortonworks.com>
Committed: Fri Jun 26 14:34:52 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/4a0c4098/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
+});