Repository: storm
Updated Branches:
  refs/heads/master e543bbf81 -> f4c6babb5


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/7852bc24
Tree: http://git-wip-us.apache.org/repos/asf/storm/tree/7852bc24
Diff: http://git-wip-us.apache.org/repos/asf/storm/diff/7852bc24

Branch: refs/heads/master
Commit: 7852bc24f051b7c5dd95835d473b99ed531e353c
Parents: 4699990
Author: Satish Duggana <[email protected]>
Authored: Wed Feb 17 11:45:33 2016 +0530
Committer: Satish Duggana <[email protected]>
Committed: Wed Feb 17 11:49:35 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/7852bc24/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 1bf85d4..510d7ab 100644
--- a/storm-core/src/clj/org/apache/storm/ui/core.clj
+++ b/storm-core/src/clj/org/apache/storm/ui/core.clj
@@ -148,8 +148,10 @@
   (logviewer-link host (Utils/eventLogsFilename topology-id port) secure?))
 
 (defn worker-log-link [host port topology-id secure?]
-  (let [fname (Utils/logsFilename 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 (Utils/logsFilename 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