You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tez.apache.org by sr...@apache.org on 2015/10/31 16:43:59 UTC

tez git commit: TEZ-2909. Tez UI: Application link in All DAGs table is disable when applicationhistory is unavailable (sree)

Repository: tez
Updated Branches:
  refs/heads/master 414258e40 -> e84d79725


TEZ-2909. Tez UI: Application link in All DAGs table is disable when applicationhistory is unavailable (sree)


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

Branch: refs/heads/master
Commit: e84d797254916cbc166cee42a2f8503b38152fd1
Parents: 414258e
Author: Sreenath Somarajapuram <sr...@apache.org>
Authored: Sat Oct 31 08:43:17 2015 -0700
Committer: Sreenath Somarajapuram <sr...@apache.org>
Committed: Sat Oct 31 08:43:17 2015 -0700

----------------------------------------------------------------------
 CHANGES.txt                                     |  1 +
 .../app/scripts/controllers/dags_controller.js  | 21 +++++---------------
 2 files changed, 6 insertions(+), 16 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tez/blob/e84d7972/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index 03ccdeb..559d748 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -7,6 +7,7 @@ INCOMPATIBLE CHANGES
   TEZ-2679. Admin forms of launch env settings
 
 ALL CHANGES:
+  TEZ-2909. Tez UI: Application link in All DAGs table is disable when applicationhistory is unavailable
   TEZ-808. Handle task attempts that are not making progress
   TEZ-2553. Tez UI: Tez UI Nits
   TEZ-2814. ATSImportTool has a return statement in a finally block

http://git-wip-us.apache.org/repos/asf/tez/blob/e84d7972/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 a99c850..2360e76 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
@@ -288,22 +288,11 @@ App.DagsController = Em.ObjectController.extend(App.PaginatedContentMixin, App.C
         templateName: 'components/basic-table/linked-cell',
         enableFilter: true,
         getCellContent: function(row) {
-          var appId = row.get('applicationId');
-          if(appId) {
-            return Em.RSVP.allSettled([
-              store.find('appDetail', appId),
-              store.find('tezApp', 'tez_' + appId)
-            ]).then(function (response) {
-              var content = {
-                displayText: row.get('applicationId'),
-                entityId: row.get('applicationId')
-              };
-              if(response.get('0.value') && response.get('1.value')) {
-                content.linkTo = 'tez-app';
-              }
-              return content;
-            });
-          }
+          return {
+            linkTo: 'tez-app',
+            entityId: row.get('applicationId'),
+            displayText: row.get('applicationId')
+          };
         }
       },
       {