Repository: storm
Updated Branches:
  refs/heads/1.x-branch cb4636c59 -> f6f05afb1


STORM-1522 should create error worker log location only when error-host and 
error-port are available


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

Branch: refs/heads/1.x-branch
Commit: b5f58d064a3bd451ebd068637f06c921daafe399
Parents: 6861ad3
Author: Satish Duggana <[email protected]>
Authored: Wed Feb 17 13:03:51 2016 +0530
Committer: Satish Duggana <[email protected]>
Committed: Wed Feb 17 13:03:51 2016 +0530

----------------------------------------------------------------------
 storm-core/src/clj/org/apache/storm/ui/core.clj | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/storm/blob/b5f58d06/storm-core/src/clj/org/apache/storm/ui/core.clj
----------------------------------------------------------------------
diff --git a/storm-core/src/clj/org/apache/storm/ui/core.clj 
b/storm-core/src/clj/org/apache/storm/ui/core.clj
index bd1dba5..9bd7a26 100644
--- a/storm-core/src/clj/org/apache/storm/ui/core.clj
+++ b/storm-core/src/clj/org/apache/storm/ui/core.clj
@@ -146,8 +146,10 @@
   (logviewer-link host (event-logs-filename topology-id port) secure?))
 
 (defn worker-log-link [host port topology-id secure?]
-  (let [fname (logs-filename topology-id port)]
-    (logviewer-link host fname secure?)))
+  (if (or (empty? host) (let [port_str (str port "")] (or (empty? port_str) (= 
"0" port_str))))
+    ""
+    (let [fname (logs-filename topology-id port)]
+      (logviewer-link host fname secure?))))
 
 (defn nimbus-log-link [host]
   (url-format "http://%s:%s/daemonlog?file=nimbus.log"; host (*STORM-CONF* 
LOGVIEWER-PORT)))

Reply via email to