From bc273c43ae9959fae224026447d61b519a4b5da4 Mon Sep 17 00:00:00 2001
From: sunilg <sunilg@apache.org>
Date: Thu, 27 Oct 2016 14:19:44 +0530
Subject: [PATCH] =?UTF-8?q?YARN-5500.=20[YARN-3368]=20=20=E2=80=98Master?=
 =?UTF-8?q?=20node'=20link=20under=20application=20tab=20is=20broken.=20(A?=
 =?UTF-8?q?khil=20P=20B=20Tan=20via=20Sunil=20G)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../src/main/webapp/app/controllers/yarn-app.js          | 9 ++++++++-
 .../src/main/webapp/app/templates/yarn-app.hbs           | 4 ++--
 2 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/app/controllers/yarn-app.js b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/app/controllers/yarn-app.js
index f6b9404d89..309c895b57 100644
--- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/app/controllers/yarn-app.js
+++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/app/controllers/yarn-app.js
@@ -33,6 +33,13 @@ export default Ember.Controller.extend({
       routeName: 'yarn-app',
       model: appId
     }];
-  })
+  }),
 
+  amHostHttpAddressFormatted: function() {
+    var amHostAddress = this.get('model.app.amHostHttpAddress');
+    if (amHostAddress.indexOf('http://') < 0) {
+      amHostAddress = 'http://' + amHostAddress;
+    }
+    return amHostAddress;
+  }.property('model.app.amHostHttpAddress')
 });
diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/app/templates/yarn-app.hbs b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/app/templates/yarn-app.hbs
index 9e92fc1672..acf00d1c3f 100644
--- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/app/templates/yarn-app.hbs
+++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/app/templates/yarn-app.hbs
@@ -172,8 +172,8 @@
 
               <tbody>
               <tr>
-                <td><a href={{model.app.amContainerLogs}}>Link</a></td>
-                <td><a href={{model.app.amHostHttpAddress}}>Link</a></td>
+                <td><a href={{model.app.amContainerLogs}} target="_blank">Link</a></td>
+                <td><a href={{amHostHttpAddressFormatted}} target="_blank">Link</a></td>
                 <td>{{model.app.amNodeLabelExpression}}</td>
               </tr>
               </tbody>