[
https://issues.apache.org/jira/browse/STORM-455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14097595#comment-14097595
]
ASF GitHub Bot commented on STORM-455:
--------------------------------------
GitHub user dan-blanchard opened a pull request:
https://github.com/apache/incubator-storm/pull/227
STORM-455 Report error-level messages from ShellBolt children
The idea here is that it'd be nice to see reported errors from `ShellBolt`
in the Storm UI, and now that we have support for multiple logging levels, we
can assume that `error` level messages are errors to be reported.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dan-blanchard/incubator-storm
shellbolt_reportfailedexception
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-storm/pull/227.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #227
----
commit 76353a80ff8d0121cdd5ef53e518f07f0be95a0a
Author: Daniel Blanchard <[email protected]>
Date: 2014-08-14T20:23:53Z
reportError when receiving an error-level message in ShellBolt.
----
> 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)
