AMBARI-4133 Perf issues on Hosts page - freezes for several seconds and then unfreezes repeatedly on a large cluster. (atkach)

ambari-web/app/controllers/global/cluster_controller.js

@@ -198,21 +198,33 @@ App.ClusterController = Em.Controller.extend({
    * Changes whenever alerts are loaded.
    */
   alerts:[],
+  alertsHostMap: {},
+  alertsServiceMap: {},
   updateAlerts: function(){
-    var alerts = App.Alert.find();
-    var alertsArray = alerts.toArray();
-    var sortedArray = alertsArray.sort(function (left, right) {
-      var statusDiff = right.get('status') - left.get('status');
-      if (statusDiff == 0) { // same error severity - sort by time
-        var rightTime = right.get('date');
-        var leftTime = left.get('date');
-        rightTime = rightTime ? rightTime.getTime() : 0;
-        leftTime = leftTime ? leftTime.getTime() : 0;
-        statusDiff = rightTime - leftTime;
+    var alerts = App.Alert.find().toArray();
+    var alertsHostMap = {};
+    var alertsServiceMap = {};
+    alerts.forEach(function (alert) {
+      if (!alert.get('isOk')) {
+        if (!alert.get('ignoredForHosts')) {
+          if (alertsHostMap[alert.get('hostName')]) {
+            alertsHostMap[alert.get('hostName')]++;
+          } else {
+            alertsHostMap[alert.get('hostName')] = 1;
+          }
+        }
+        if (!alert.get('ignoredForServices')) {
+          if (alertsServiceMap[alert.get('serviceType')]) {
+            alertsServiceMap[alert.get('serviceType')]++;
+          } else {
+            alertsServiceMap[alert.get('serviceType')] = 1;
+          }
+        }
       }
-      return statusDiff;
-    });
-    this.set('alerts', sortedArray);
+    }, this);
+    this.set('alertsHostMap', alertsHostMap);
+    this.set('alertsServiceMap', alertsServiceMap);
+    this.set('alerts', alerts);
   },
 
   /**

ambari-web/app/controllers/main/host.js

@@ -87,7 +87,7 @@ App.MainHostController = Em.ArrayController.extend({
         hostAlerts: function () {
           var allAlerts = App.router.get('clusterController.alerts').filterProperty('ignoredForHosts', false);
           if (host) {
-            return allAlerts.filterProperty('hostName', host.get('hostName'));
+            return App.Alert.sort(allAlerts.filterProperty('hostName', host.get('hostName')));
           }
           return 0;
         }.property('App.router.clusterController.alerts'),

ambari-web/app/controllers/main/service/info/summary.js

@@ -26,9 +26,9 @@ App.MainServiceInfoSummaryController = Em.Controller.extend({
   alerts: function () {
     var serviceId = this.get('content.serviceName');
     if (serviceId) {
-      return this.get('allAlerts').filter(function (item) {
+      return App.Alert.sort(this.get('allAlerts').filter(function (item) {
         return item.get('serviceType').toLowerCase() == serviceId.toLowerCase() && !item.get('ignoredForServices');
-      });
+      }));
     }
     return [];
   }.property('allAlerts', 'content.serviceName'),

ambari-web/app/mappers/alerts_mapper.js

@@ -69,16 +69,18 @@ App.alertsMapper = App.QuickDataMapper.create({
       }, this);
 
       this.get('model').find().forEach(function (alertRecord) {
-        var existAlert = alertsMap[alertRecord.get('id')];
-        if (existAlert) {
-          existAlert = this.getDiscrepancies(existAlert, previousAlertsResponse[alertRecord.get('id')], mutableFields);
+        if (alertRecord) {
+          var existAlert = alertsMap[alertRecord.get('id')];
           if (existAlert) {
-            for (var i in existAlert) {
-              alertRecord.set(stringUtils.underScoreToCamelCase(i), existAlert[i]);
+            existAlert = this.getDiscrepancies(existAlert, previousAlertsResponse[alertRecord.get('id')], mutableFields);
+            if (existAlert) {
+              for (var i in existAlert) {
+                alertRecord.set(stringUtils.underScoreToCamelCase(i), existAlert[i]);
+              }
             }
+          } else {
+            this.deleteRecord(alertRecord);
           }
-        } else {
-          this.deleteRecord(alertRecord);
         }
       }, this);
 

ambari-web/app/models/alert.js

@@ -189,5 +189,19 @@ App.Alert = DS.Model.extend({
 
 });
 
+App.Alert.sort = function (array) {
+  return array.sort(function (left, right) {
+    var statusDiff = right.get('status') - left.get('status');
+    if (statusDiff == 0) { // same error severity - sort by time
+      var rightTime = right.get('date');
+      var leftTime = left.get('date');
+      rightTime = rightTime ? rightTime.getTime() : 0;
+      leftTime = leftTime ? leftTime.getTime() : 0;
+      statusDiff = rightTime - leftTime;
+    }
+    return statusDiff;
+  });
+};
+
 App.Alert.FIXTURES = [
 ];

ambari-web/app/models/host.js

@@ -58,7 +58,7 @@ App.Host = DS.Model.extend({
   }.property('memTotal', 'memFree'),
 
   criticalAlertsCount: function () {
-    return App.router.get('clusterController.alerts').filterProperty('hostName', this.get('hostName')).filterProperty('isOk', false).filterProperty('ignoredForHosts', false).length;
+    return App.router.get('clusterController.alertsHostMap')[this.get('hostName')];
   }.property('App.router.clusterController.alerts.length'),
 
   componentsWithStaleConfigsCount: function() {

ambari-web/app/views/main/dashboard.js

@@ -546,7 +546,7 @@ App.MainDashboardView = Em.View.extend({
           var allAlerts = App.router.get('clusterController.alerts');
           var serviceId = this.get('service.serviceName');
           if (serviceId) {
-            return allAlerts.filterProperty('serviceType', serviceId).filterProperty('isOk', false).filterProperty('ignoredForServices', false);
+            return App.Alert.sort(allAlerts.filterProperty('serviceType', serviceId).filterProperty('isOk', false).filterProperty('ignoredForServices', false));
           }
           return 0;
         }.property('App.router.clusterController.alerts'),

ambari-web/app/views/main/dashboard/service.js

@@ -123,8 +123,7 @@ App.MainDashboardServiceView = Em.View.extend({
   },
 
   criticalAlertsCount: function () {
-    var alerts = App.router.get('clusterController.alerts');
-    return alerts.filterProperty('serviceType', this.get('service.id')).filterProperty('isOk', false).filterProperty('ignoredForServices', false).length;
+    return App.router.get('clusterController.alertsServiceMap')[this.get('service.id')];
   }.property('App.router.clusterController.alerts'),
 
   isCollapsed: false,