[GitHub] flink pull request #4376: [FLINK-6521] Add per job cleanup methods to HighAv...

2017-07-28 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/4376#discussion_r130052646 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/highavailability/zookeeper/HighAvailabilityServiceTest.java --- @@ -0,0 +1,115 @@

[GitHub] flink pull request #4376: [FLINK-6521] Add per job cleanup methods to HighAv...

2017-07-28 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/4376#discussion_r130052278 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/highavailability/zookeeper/HighAvailabilityServiceTest.java --- @@ -0,0 +1,115 @@

[GitHub] flink pull request #4376: [FLINK-6521] Add per job cleanup methods to HighAv...

2017-07-28 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/4376#discussion_r130052071 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/highavailability/zookeeper/ZooKeeperHaServices.java --- @@ -113,6 +116,11 @@ public

[GitHub] flink pull request #4376: [FLINK-6521] Add per job cleanup methods to HighAv...

2017-07-20 Thread zjureel
GitHub user zjureel opened a pull request: https://github.com/apache/flink/pull/4376 [FLINK-6521] Add per job cleanup methods to HighAvailabilityServices Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into