Repository: tez
Updated Branches:
  refs/heads/master d932579b0 -> bfb34afba


TEZ-2329. UI Query on final dag status performance improvement (jeagles)


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

Branch: refs/heads/master
Commit: bfb34afba0edfb254b05037b3b2ab37e3d3e44cf
Parents: d932579
Author: Jonathan Eagles <[email protected]>
Authored: Thu Apr 16 08:57:17 2015 -0500
Committer: Jonathan Eagles <[email protected]>
Committed: Thu Apr 16 08:57:17 2015 -0500

----------------------------------------------------------------------
 CHANGES.txt                                                    | 1 +
 .../src/main/webapp/app/scripts/controllers/dags_controller.js | 6 +++++-
 tez-ui/src/main/webapp/app/scripts/helpers/misc.js             | 4 ++++
 3 files changed, 10 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tez/blob/bfb34afb/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index 11c7322..a41a724 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -121,6 +121,7 @@ Release 0.6.1: Unreleased
 INCOMPATIBLE CHANGES
 
 ALL CHANGES:
+  TEZ-2329. UI Query on final dag status performance improvement
   TEZ-2287. Deprecate VertexManagerPluginContext.getTaskContainer().
   TEZ-1909. Remove need to copy over all events from attempt 1 to attempt 2 dir
   TEZ-2061. Tez UI: vertex id column and filter on tasks page should be 
changed to vertex name

http://git-wip-us.apache.org/repos/asf/tez/blob/bfb34afb/tez-ui/src/main/webapp/app/scripts/controllers/dags_controller.js
----------------------------------------------------------------------
diff --git a/tez-ui/src/main/webapp/app/scripts/controllers/dags_controller.js 
b/tez-ui/src/main/webapp/app/scripts/controllers/dags_controller.js
index 9968a6a..f6a7013 100644
--- a/tez-ui/src/main/webapp/app/scripts/controllers/dags_controller.js
+++ b/tez-ui/src/main/webapp/app/scripts/controllers/dags_controller.js
@@ -55,9 +55,13 @@ App.DagsController = 
Em.ObjectController.extend(App.PaginatedContentMixin, App.C
         user: this.user_filter
       },
       secondary: {
-        status: this.status_filter
       }
     }
+    if (App.Helpers.misc.isFinalDagStatus(this.status_filter)) {
+      filters.primary['status'] = this.status_filter;
+    } else {
+      filters.secondary['status'] = this.status_filter;
+    }
     this.setFiltersAndLoadEntities(filters);
   },
 

http://git-wip-us.apache.org/repos/asf/tez/blob/bfb34afb/tez-ui/src/main/webapp/app/scripts/helpers/misc.js
----------------------------------------------------------------------
diff --git a/tez-ui/src/main/webapp/app/scripts/helpers/misc.js 
b/tez-ui/src/main/webapp/app/scripts/helpers/misc.js
index 53149a4..6794e95 100644
--- a/tez-ui/src/main/webapp/app/scripts/helpers/misc.js
+++ b/tez-ui/src/main/webapp/app/scripts/helpers/misc.js
@@ -90,6 +90,10 @@ App.Helpers.misc = {
       'KILLED', 'FAILED', 'ERROR']) != -1;
   },
 
+  isFinalDagStatus: function(status) {
+    return $.inArray(status, ['SUCCEEDED', 'KILLED', 'FAILED', 'ERROR']) != -1;
+  },
+
   isValidTaskStatus: function(status) {
     return $.inArray(status, ['RUNNING', 'SUCCEEDED', 'FAILED', 'KILLED']) != 
-1;
   },

Reply via email to