[GitHub] flink pull request: [FLINK-2329] [runtime] Introduces InstanceGate...

2015-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/893 --- 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] flink pull request: [FLINK-2329] [runtime] Introduces InstanceGate...

2015-07-14 Thread uce
Github user uce commented on the pull request: https://github.com/apache/flink/pull/893#issuecomment-121238472 Very nice for testing! *happy* --- 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 t

[GitHub] flink pull request: [FLINK-2329] [runtime] Introduces InstanceGate...

2015-07-13 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/893#issuecomment-120938193 Merging this... --- 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 fea

[GitHub] flink pull request: [FLINK-2329] [runtime] Introduces InstanceGate...

2015-07-09 Thread mxm
Github user mxm commented on the pull request: https://github.com/apache/flink/pull/893#issuecomment-120018025 Very sensible change in anticipation of the upcoming high availability changes. +1 --- If your project is set up for it, you can reply to this email and have your r

[GitHub] flink pull request: [FLINK-2329] [runtime] Introduces InstanceGate...

2015-07-09 Thread mxm
Github user mxm commented on a diff in the pull request: https://github.com/apache/flink/pull/893#discussion_r34264492 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/instance/Instance.java --- @@ -43,8 +41,8 @@ /** The lock on which to synchronize allocation

[GitHub] flink pull request: [FLINK-2329] [runtime] Introduces InstanceGate...

2015-07-08 Thread aljoscha
Github user aljoscha commented on the pull request: https://github.com/apache/flink/pull/893#issuecomment-119691808 +1 The idea seems very reasonable. --- 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 n

[GitHub] flink pull request: [FLINK-2329] [runtime] Introduces InstanceGate...

2015-07-08 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/893#issuecomment-119688164 Good patch! +1 to merge, from my side --- 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

[GitHub] flink pull request: [FLINK-2329] [runtime] Introduces InstanceGate...

2015-07-08 Thread tillrohrmann
GitHub user tillrohrmann opened a pull request: https://github.com/apache/flink/pull/893 [FLINK-2329] [runtime] Introduces InstanceGateway for RPCs from Executions This PR replaces the direct `ActorRef` interaction from within `Executions` with an `InstanceGateway`. The `InstanceGat