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/08/21 14:22:40 UTC

ambari git commit: AMBARI-12843. Disable Ranger HDFS plugin shows dfs property update which is incorrect (alexantonenko)

Repository: ambari
Updated Branches:
  refs/heads/trunk eadde3d58 -> 762f7d9e4


AMBARI-12843. Disable Ranger HDFS plugin shows dfs property update which is incorrect (alexantonenko)


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

Branch: refs/heads/trunk
Commit: 762f7d9e47479fa2b19fd23a7593d57a117052c0
Parents: eadde3d
Author: Alex Antonenko <hi...@gmail.com>
Authored: Fri Aug 21 09:46:23 2015 +0300
Committer: Alex Antonenko <hi...@gmail.com>
Committed: Fri Aug 21 15:22:35 2015 +0300

----------------------------------------------------------------------
 .../app/utils/configs/modification_handlers/hdfs.js  | 15 ---------------
 1 file changed, 15 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/762f7d9e/ambari-web/app/utils/configs/modification_handlers/hdfs.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/utils/configs/modification_handlers/hdfs.js b/ambari-web/app/utils/configs/modification_handlers/hdfs.js
index ad97e22..c77a716 100644
--- a/ambari-web/app/utils/configs/modification_handlers/hdfs.js
+++ b/ambari-web/app/utils/configs/modification_handlers/hdfs.js
@@ -27,7 +27,6 @@ module.exports = App.ServiceConfigModificationHandler.create({
     var rangerPluginEnabledName = "ranger-hdfs-plugin-enabled";
     var affectedPropertyName = changedConfig.get("name");
     if (App.get('isHadoop23Stack') && affectedPropertyName == rangerPluginEnabledName) {
-      var configDfsPermissionsEnabled = this.getConfig(allConfigs, 'dfs.permissions.enabled', 'hdfs-site.xml', 'HDFS');
       var configAttributesProviderClass = this.getConfig(allConfigs, 'dfs.namenode.inode.attributes.provider.class', 'hdfs-site.xml', 'HDFS');
       var isAttributesProviderClassSet = typeof configAttributesProviderClass !== 'undefined';
 
@@ -35,20 +34,6 @@ module.exports = App.ServiceConfigModificationHandler.create({
       var newDfsPermissionsEnabled = rangerPluginEnabled ? "true" : "false";
       var newAttributesProviderClass = 'org.apache.ranger.authorization.hadoop.RangerHdfsAuthorizer';
 
-      // Add HDFS-Ranger configs
-      if (configDfsPermissionsEnabled != null && newDfsPermissionsEnabled !== configDfsPermissionsEnabled.get('value')) {
-        affectedProperties.push({
-          serviceName : "HDFS",
-          sourceServiceName : "HDFS",
-          propertyName : 'dfs.permissions.enabled',
-          propertyDisplayName : 'dfs.permissions.enabled',
-          newValue : newDfsPermissionsEnabled,
-          curValue : configDfsPermissionsEnabled.get('value'),
-          changedPropertyName : rangerPluginEnabledName,
-          removed : false,
-          filename : 'hdfs-site.xml'
-        });
-      }
       if (rangerPluginEnabled && (!isAttributesProviderClassSet || newAttributesProviderClass != configAttributesProviderClass.get('value'))) {
         affectedProperties.push({
           serviceName : "HDFS",