[GitHub] storm pull request: Kafka spout failure config

2015-02-24 Thread ogorun
Github user ogorun commented on the pull request: https://github.com/apache/storm/pull/406#issuecomment-75769012 Hi @d2r , Is it better to open separate JIRA issue or this pull request is considered as part of subjects discussed in STORM-650? --- If your project is set up

[GitHub] storm pull request: Kafka spout failure config

2015-02-03 Thread ogorun
Github user ogorun commented on the pull request: https://github.com/apache/storm/pull/406#issuecomment-72620333 @wurstmeister , Thank you for pointing me to https://github.com/apache/storm/pull/254 . It is definitely progress in direction of failure handling functionality

[GitHub] storm pull request: Storm 590: added Kafka API based partition rea...

2015-02-02 Thread ogorun
Github user ogorun commented on the pull request: https://github.com/apache/storm/pull/407#issuecomment-72506514 Hi @wurstmeister, There is my pull request https://github.com/apache/storm/pull/406 and corresponding comments in pull request https://github.com/apache/storm

[GitHub] storm pull request: Storm 631: refactoring kafka connector code.

2015-02-02 Thread ogorun
Github user ogorun commented on the pull request: https://github.com/apache/storm/pull/387#issuecomment-72505185 Hi @Parth-Brahmbhatt , Thank you for pointing me to this pull request. I see that my pull request https://github.com/apache/storm/pull/406 shares part of ideas

[GitHub] storm pull request: Kafka spout failure config

2015-02-02 Thread ogorun
Github user ogorun commented on the pull request: https://github.com/apache/storm/pull/406#issuecomment-72471145 Thank you for feedback. I'll look at it --- 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 pr

[GitHub] storm pull request: Kafka spout failure config

2015-02-02 Thread ogorun
GitHub user ogorun opened a pull request: https://github.com/apache/storm/pull/406 Kafka spout failure config This pull request offers plugable MessageFailureHandler for failed message. Solution structure. PartitionManger uses instance of class that implements