You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by at...@apache.org on 2014/05/29 16:54:30 UTC

git commit: AMBARI-5927 Manage Configuration groups takes too long to load hosts on 2K node cluster. (atkach)

Repository: ambari
Updated Branches:
  refs/heads/trunk ff2b111b1 -> a384a0e1c


AMBARI-5927 Manage Configuration groups takes too long to load hosts on 2K node cluster. (atkach)


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

Branch: refs/heads/trunk
Commit: a384a0e1c4cca0673250ff0cf492e7af942cd64d
Parents: ff2b111
Author: atkach <at...@hortonworks.com>
Authored: Thu May 29 17:43:26 2014 +0300
Committer: atkach <at...@hortonworks.com>
Committed: Thu May 29 17:43:26 2014 +0300

----------------------------------------------------------------------
 ambari-web/app/models/config_group.js       | 2 +-
 ambari-web/test/models/config_group_test.js | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/a384a0e1/ambari-web/app/models/config_group.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/models/config_group.js b/ambari-web/app/models/config_group.js
index b1f178b..82db2a9 100644
--- a/ambari-web/app/models/config_group.js
+++ b/ambari-web/app/models/config_group.js
@@ -122,7 +122,7 @@ App.ConfigGroup = Ember.Object.extend({
     }
     // parentConfigGroup.hosts(hosts from default group) - are available hosts, which don't belong to any group
     this.get('parentConfigGroup.hosts').forEach(function (hostName) {
-      unusedHostsMap[hostName.get('id')] = true;
+      unusedHostsMap[hostName] = true;
     });
     sharedHosts.forEach(function (host) {
       if (unusedHostsMap[host.get('id')]) {

http://git-wip-us.apache.org/repos/asf/ambari/blob/a384a0e1/ambari-web/test/models/config_group_test.js
----------------------------------------------------------------------
diff --git a/ambari-web/test/models/config_group_test.js b/ambari-web/test/models/config_group_test.js
index 9c5b09f..6ea19c7 100644
--- a/ambari-web/test/models/config_group_test.js
+++ b/ambari-web/test/models/config_group_test.js
@@ -50,7 +50,7 @@ var configGroup,
   ],
   setParentConfigGroup = function (configGroup, hosts) {
     configGroup.set('parentConfigGroup', App.ConfigGroup.create());
-    configGroup.set('parentConfigGroup.hosts', hosts);
+    configGroup.set('parentConfigGroup.hosts', hosts.mapProperty('hostName'));
   };
 
 describe('App.ConfigGroup', function () {