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

2016-03-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/407 --- 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 enabl

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

2016-03-07 Thread knusbaum
Github user knusbaum commented on the pull request: https://github.com/apache/storm/pull/407#issuecomment-193478032 Closing --- 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 enable

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

2015-06-15 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/storm/pull/407#issuecomment-112219527 @wurstmeister What is the status of this PR? I see it needs an up-merge, but other than that, is there any other work that this needs before we review it? --- If your proj

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

2015-02-04 Thread wurstmeister
Github user wurstmeister commented on the pull request: https://github.com/apache/storm/pull/407#issuecomment-72850441 this is going to be discussed in STORM-650 --- 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 pro

[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/issu

[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