[
https://issues.apache.org/jira/browse/STORM-455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14102421#comment-14102421
]
ASF GitHub Bot commented on STORM-455:
--------------------------------------
Github user revans2 commented on the pull request:
https://github.com/apache/incubator-storm/pull/227#issuecomment-52660775
I think this is fine, we have rate limiting so we should not overwhelm ZK
too badly.
+1, I'll merge this in
> Error-level messages from ShellBolts should turn into ReportedFailedExceptions
> ------------------------------------------------------------------------------
>
> Key: STORM-455
> URL: https://issues.apache.org/jira/browse/STORM-455
> Project: Apache Storm (Incubating)
> Issue Type: Improvement
> Affects Versions: 0.9.3-incubating
> Reporter: Dan Blanchard
> Fix For: 0.9.3-incubating
>
>
> Now that support was added for logging levels in the multilang protocol, it
> would be really nice if those of us writing Bolts in other languages had the
> ability to raise {{ReportedFailedException}} so that errors show up in the
> Storm UI.
> I propose that any time an {{error}} level logging message is sent to a
> ShellBolt, it raises a {{ReportedFailedException}}.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
