AMBARI-1494. Browser memory consumption issues. (jaimin)

git-svn-id: https://svn.apache.org/repos/asf/incubator/ambari/trunk@1449954 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -374,6 +374,8 @@ Trunk (unreleased changes):
 
  BUG FIXES
 
+ AMBARI-1494. Browser memory consumption issues. (jaimin)
+
  AMBARI-1480. Comparison predicate should account for null values. (tbeerbower)
 
  AMBARI-1467. UI should block on cluster metric api call before making

ambari-web/app/config.js

@@ -35,6 +35,7 @@ App.bgOperationsUpdateInterval = 6000;
 App.componentsUpdateInterval = 6000;
 App.contentUpdateInterval = 15000;
 App.maxRunsForAppBrowser = 500;
+App.pageReloadTime=3600000;
 
 // this is to make sure that IE does not cache data when making AJAX calls to the server
 $.ajaxSetup({

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

@@ -186,6 +186,7 @@ App.ClusterController = Em.Controller.extend({
    * property, which will trigger the alerts property.
    */
   loadAlerts:function () {
+    var self=this;
     if(App.router.get('updateController.isUpdated')){
       return;
     }
@@ -200,8 +201,8 @@ App.ClusterController = Em.Controller.extend({
         dataType:"json",
         context:this,
         complete:function (jqXHR, textStatus) {
-          this.updateLoadStatus('alerts');
-          this.updateAlerts();
+          self.updateLoadStatus('alerts');
+          self.updateAlerts();
         },
         error: function(jqXHR, testStatus, error) {
           // this.showMessage(Em.I18n.t('nagios.alerts.unavailable'));

ambari-web/app/controllers/main.js

@@ -52,5 +52,19 @@ App.MainController = Em.Controller.extend({
     App.router.get('updateController').set('isWorking', false);
     App.router.get('backgroundOperationsController').set('isWorking', false);
     App.router.get('clusterController').set('isWorking', false);
-  }
+  },
+
+  reloadTimeOut: null,
+
+  pageReload: function () {
+
+    clearTimeout(this.get("reloadTimeOut"));
+
+    this.set('reloadTimeOut',
+        setTimeout(function () {
+          location.reload()
+        }, App.pageReloadTime)
+    );
+  }.observes("App.router.location.lastSetURL")
+
 })

ambari-web/app/utils/http_client.js

@@ -38,6 +38,9 @@ App.HttpClient = Em.Object.create({
     }
   },
 
+  emptyFunc: function () {
+  },
+
   /**
    * @param {string} url
    * @param {Object} ajaxOptions
@@ -51,22 +54,41 @@ App.HttpClient = Em.Object.create({
       errorHandler = this.defaultErrorHandler;
     }
 
-    var options = {dataType: 'json'}; // default ajax options;
+    var xhr = new XMLHttpRequest();
+    var curTime = new Date().getTime();
 
-    $.extend(options, {
-      url: url,
-      success: function (data) {
-        try {
-          App.store.commit();
-        } catch (err) {}
-        mapper.map(data);
-      },
-      error: errorHandler
-    });
+    xhr.open('GET', url + "?_=" + curTime, true);
+    xhr.send(null);
 
-    $.extend(options, ajaxOptions);
+    this.onReady(xhr, "", ajaxOptions, mapper, errorHandler);
+  },
 
-    $.ajax(options);
+  /*
+   This function checks if we get response from server
+   Not using onreadystatechange cuz of possible closure
+   */
+  onReady: function (xhr, tm, tmp_val, mapper, errorHandler) {
+    var self = this;
+    clearTimeout(tm);
+    var timeout = setTimeout(function () {
+      if (xhr.readyState == 4) {
+        if (xhr.status == 200) {
+          mapper.map($.parseJSON(xhr.responseText));
+          tmp_val.complete.call(self);
+          xhr.abort();
+        } else {
+          errorHandler(xhr , "error", xhr.statusText);
+        }
+
+        tmp_val = null;
+        xhr = self.emptyFunc();
+        clearTimeout(timeout);
+        timeout = null;
+
+      } else {
+        self.onReady(xhr, timeout, tmp_val, mapper, errorHandler);
+      }
+    }, 10);
   },
 
   /**
@@ -78,9 +100,16 @@ App.HttpClient = Em.Object.create({
    */
   get: function (url, mapper, data, errorHandler, interval) {
     var eHandler = data.complete
+    if (!errorHandler && data.error) {
+      errorHandler = data.error;
+    }
     var client = this;
     var request = function () {
       client.request(url, data, mapper, errorHandler);
+      url=null;
+      data=null;
+      mapper=null;
+      errorHandler=null;
     }
 
     interval = "" + interval;
@@ -113,7 +142,7 @@ App.HttpClient = Em.Object.create({
 });
 
 /*App.HttpClient.get(
-  'http://nagiosserver/hdp/nagios/nagios_alerts.php?q1=alerts&alert_type=all',
-  App.alertsMapper,
-  { dataType: 'jsonp', jsonp: 'jsonp' }
-);*/
+ 'http://nagiosserver/hdp/nagios/nagios_alerts.php?q1=alerts&alert_type=all',
+ App.alertsMapper,
+ { dataType: 'jsonp', jsonp: 'jsonp' }
+ );*/