Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5701#discussion_r175731826
  
    --- Diff: 
flink-tests/src/test/java/org/apache/flink/test/checkpointing/utils/SavepointMigrationTestBase.java
 ---
    @@ -194,49 +187,35 @@ protected final void restoreAndExecute(
                        String savepointPath,
                        Tuple2<String, Integer>... expectedAccumulators) throws 
Exception {
     
    -           // Retrieve the job manager
    -           Await.result(cluster.leaderGateway().future(), 
DEADLINE.timeLeft());
    +           ClusterClient<?> client = 
miniClusterResource.getClusterClient();
    +           client.setDetached(true);
    --- End diff --
    
    As a side note which is out of scope for this issue. I think we should 
`deprecate` `ClusterClient#setDetached`. It should not be an attribute of the 
client but more of how you submit a job.


---

Reply via email to