[ 
https://issues.apache.org/jira/browse/STORM-360?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14039145#comment-14039145
 ] 

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_r14034403
  
    --- Diff: storm-core/src/clj/backtype/storm/ui/core.clj ---
    @@ -524,7 +547,11 @@
             :let [stats-seq (get-filled-stats summs)
                   stats (aggregate-spout-streams
                          (aggregate-spout-stats
    -                      stats-seq include-sys?))]]
    +                      stats-seq include-sys?))
    +              last-error (most-recent-error (get errors id))
    +              error-host (get-error-host last-error)
    +              error-port (get-error-port last-error error-host top-id)
    +              ]]
    --- End diff --
    
    Please put the closing ']' on the line above.


> 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)

Reply via email to