[GitHub] flink pull request #5775: [FLINK-9099] Assign Execution to LogicalSlot when ...

2018-03-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/5775 ---

[GitHub] flink pull request #5775: [FLINK-9099] Assign Execution to LogicalSlot when ...

2018-03-28 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/5775#discussion_r177726389 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/executiongraph/ExecutionGraphSchedulingTest.java --- @@ -465,6 +464,58 @@ public void

[GitHub] flink pull request #5775: [FLINK-9099] Assign Execution to LogicalSlot when ...

2018-03-28 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/5775#discussion_r177699910 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/jobmaster/slotpool/SlotPool.java --- @@ -659,7 +660,7 @@ public void

[GitHub] flink pull request #5775: [FLINK-9099] Assign Execution to LogicalSlot when ...

2018-03-27 Thread GJL
Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/5775#discussion_r177544954 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/executiongraph/ExecutionGraphSchedulingTest.java --- @@ -465,6 +464,58 @@ public void

[GitHub] flink pull request #5775: [FLINK-9099] Assign Execution to LogicalSlot when ...

2018-03-27 Thread GJL
Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/5775#discussion_r177544459 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/executiongraph/ExecutionGraphSchedulingTest.java --- @@ -465,6 +464,58 @@ public void

[GitHub] flink pull request #5775: [FLINK-9099] Assign Execution to LogicalSlot when ...

2018-03-27 Thread GJL
Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/5775#discussion_r177544274 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/executiongraph/ExecutionGraphSchedulingTest.java --- @@ -465,6 +464,58 @@ public void

[GitHub] flink pull request #5775: [FLINK-9099] Assign Execution to LogicalSlot when ...

2018-03-27 Thread GJL
Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/5775#discussion_r177543886 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/jobmaster/slotpool/SlotPool.java --- @@ -659,7 +660,7 @@ public void disconnectResourceManager()

[GitHub] flink pull request #5775: [FLINK-9099] Assign Execution to LogicalSlot when ...

2018-03-27 Thread tillrohrmann
GitHub user tillrohrmann opened a pull request: https://github.com/apache/flink/pull/5775 [FLINK-9099] Assign Execution to LogicalSlot when slot is assigned to Execution ## What is the purpose of the change In order to fail fast if an allocated slot is released by the