You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by dossett <gi...@git.apache.org> on 2015/06/30 16:35:37 UTC

[GitHub] storm pull request: STORM-799 Use IErrorReport interface more broa...

GitHub user dossett opened a pull request:

    https://github.com/apache/storm/pull/607

    STORM-799 Use IErrorReport interface more broadly

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/dossett/storm STORM-799

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/607.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 #607
    
----
commit 735fae7899eb8240b7a0d5514c6f68a9f6be3e4f
Author: Aaron Dossett <aa...@target.com>
Date:   2015-06-30T14:29:32Z

    STORM-799 Use IErrorReport interface more broadly

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm pull request: STORM-799 Use IErrorReport interface more broa...

Posted by revans2 <gi...@git.apache.org>.
Github user revans2 commented on the pull request:

    https://github.com/apache/storm/pull/607#issuecomment-122006252
  
    +1.  The only possible issue is if someone is implementing their own OutputCollector, possibly for running storm topologies on a different runtime. And I am not too concerned about breaking them. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm pull request: STORM-799 Use IErrorReport interface more broa...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/storm/pull/607


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---