[
https://issues.apache.org/jira/browse/STORM-360?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14039144#comment-14039144
]
ASF GitHub Bot commented on STORM-360:
--------------------------------------
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/incubator-storm/pull/150#discussion_r14034358
--- Diff: storm-core/src/clj/backtype/storm/ui/core.clj ---
@@ -291,6 +290,30 @@
(map nil-to-zero)
(apply max)))
+(defn get-error-span [error]
--- End diff --
For these new functions could we follow the new convention with the
parameters on the second line and put the closing ')' on the line with the last
code.
> Add woker-id/host info/port info to the error for easy tracking on UI
> ---------------------------------------------------------------------
>
> Key: STORM-360
> URL: https://issues.apache.org/jira/browse/STORM-360
> Project: Apache Storm (Incubating)
> Issue Type: Bug
> Reporter: Kishor Patil
> Labels: master
>
> As the ErrorInfo get from nimbus has no detailed information of where did
> this error happen. Can task id host/port for can be added to the error
> information? It can easy storm debug. Currently we also can only see the
> error information in UI but cannot which component throws this exception and
> track it to logs.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
