[GitHub] flink pull request: [FLINK-2291] [runtime] Adds high availability ...

2015-08-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/1016 --- 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 enab

[GitHub] flink pull request: [FLINK-2291] [runtime] Adds high availability ...

2015-08-31 Thread tillrohrmann
Github user tillrohrmann commented on the pull request: https://github.com/apache/flink/pull/1016#issuecomment-136327681 Failing test is a Yarn test. Locally Travis passed. Will merge it now. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] flink pull request: [FLINK-2291] [runtime] Adds high availability ...

2015-08-28 Thread tillrohrmann
Github user tillrohrmann commented on the pull request: https://github.com/apache/flink/pull/1016#issuecomment-135781842 Ok, thanks for your feedback @StephanEwen and @uce. I'll rebase on the latest master and then merge it. I've created two issues [1, 2] to track the problems Stephan

[GitHub] flink pull request: [FLINK-2291] [runtime] Adds high availability ...

2015-08-25 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1016#issuecomment-134647030 +1 to merge, we should follow up on the Mini cluster and Curator shading separately --- If your project is set up for it, you can reply to this email and have your

[GitHub] flink pull request: [FLINK-2291] [runtime] Adds high availability ...

2015-08-25 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1016#issuecomment-134646755 Looks very good. Minor comments that we may address after this pull request: - The Flink Mini cluster becomes tricky, the configurations ever more intransp

[GitHub] flink pull request: [FLINK-2291] [runtime] Adds high availability ...

2015-08-19 Thread uce
Github user uce commented on the pull request: https://github.com/apache/flink/pull/1016#issuecomment-132504227 Very nice, Till. I will rebase my changes for FLINK-2354 on this. Your description in this PR and the docs were already very helpful in this regard. I got delayed la

[GitHub] flink pull request: [FLINK-2291] [runtime] Adds high availability ...

2015-08-14 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1016#issuecomment-131249089 From a first glance this looks super nice! Very excited to get this in and try it out seriously on a cluster! Good day, today :-) --- If your project is se

[GitHub] flink pull request: [FLINK-2291] [runtime] Adds high availability ...

2015-08-14 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1016#issuecomment-131246273 Big piece of work. I'd like to have a look at this, but it may take a few days... --- If your project is set up for it, you can reply to this email and have

[GitHub] flink pull request: [FLINK-2291] [runtime] Adds high availability ...

2015-08-14 Thread tillrohrmann
GitHub user tillrohrmann opened a pull request: https://github.com/apache/flink/pull/1016 [FLINK-2291] [runtime] Adds high availability support via ZooKeeper ## Idea This PR introduces cluster high availability via ZooKeeper. The idea is to use ZooKeeper to do leader electi