[tez] branch master updated: TEZ-4188. Link to NodeManager Logs of Home and DAG details doesn't consider yarnProtocol

[je...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jeagles pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tez.git


The following commit(s) were added to refs/heads/master by this push:
     new 8364f17  TEZ-4188. Link to NodeManager Logs of Home and DAG details doesn't consider yarnProtocol
8364f17 is described below

commit 8364f17abd9a6adc95e100bb308b22754529f8f8
Author: Eugene Chung <eu...@navercorp.com>
AuthorDate: Fri Aug 7 16:27:22 2020 -0500

    TEZ-4188. Link to NodeManager Logs of Home and DAG details doesn't consider yarnProtocol
    
    Signed-off-by: Jonathan Eagles <je...@apache.org>
---
 tez-ui/src/main/webapp/app/serializers/dag.js | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tez-ui/src/main/webapp/app/serializers/dag.js b/tez-ui/src/main/webapp/app/serializers/dag.js
index de60985..701c1d2 100644
--- a/tez-ui/src/main/webapp/app/serializers/dag.js
+++ b/tez-ui/src/main/webapp/app/serializers/dag.js
@@ -75,8 +75,9 @@ function getContainerLogs(source) {
   for (var key in otherinfo) {
     if (key.indexOf('inProgressLogsURL_') === 0) {
       let logs = Ember.get(source, 'otherinfo.' + key);
-      if (logs.indexOf('http') !== 0) {
-        logs = 'http://' + logs;
+      if (logs.indexOf("://") === -1) {
+        let yarnProtocol = this.get('env.app.yarnProtocol');
+        logs = yarnProtocol + '://' + logs;
       }
       let attemptID = key.substring(18);
       containerLogs.push({