[GitHub] storm pull request: Kafka spout failure config

2015-02-02 Thread wurstmeister
Github user wurstmeister commented on the pull request: https://github.com/apache/storm/pull/406#issuecomment-72532975 Hi, Thanks for pointing out your PR. This looks like it is fairly similar to https://github.com/apache/storm/pull/254, which has been merged recently

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

2015-02-02 Thread wurstmeister
Github user wurstmeister commented on the pull request: https://github.com/apache/storm/pull/387#issuecomment-72537785 sounds good! --- 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

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

2015-02-02 Thread wurstmeister
GitHub user wurstmeister opened a pull request: https://github.com/apache/storm/pull/407 Storm 590: added Kafka API based partition reader Following the discussion in https://github.com/apache/storm/pull/387 I've prepared an alternative implementation that is backwards compatible

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

2015-02-02 Thread wurstmeister
Github user wurstmeister commented on the pull request: https://github.com/apache/storm/pull/387#issuecomment-72496938 Hi, i've created a backwards compatible alternative focused on using the Kafka API for reading partition information in https://github.com/apache/storm

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

2015-01-31 Thread wurstmeister
Github user wurstmeister commented on the pull request: https://github.com/apache/storm/pull/387#issuecomment-72336141 Hi thanks for starting this refactoring, i've had this on my list of things to do but unfortunately have not had the time to do it. A few