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

    https://github.com/apache/flink/pull/5487#discussion_r169913228
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/jobmaster/JobMaster.java 
---
    @@ -963,6 +1120,36 @@ private Acknowledge suspendExecution(final Exception 
cause) {
                return Acknowledge.get();
        }
     
    +   /**
    +    * Schedules the execution of the current {@link ExecutionGraph}.
    +    */
    +   private void scheduleExecutionGraph() {
    +           try {
    +                   executionGraph.scheduleForExecution();
    +           }
    +           catch (Throwable t) {
    +                   executionGraph.failGlobal(t);
    +           }
    +   }
    +
    +   /**
    +    * Dispose the savepoint stored under the given path.
    +    *
    +    * @param savepointPath path where the savepoint is stored
    +    */
    +   private void disposeSavepoint(String savepointPath) {
    +           try {
    +                   // delete the temporary savepoint
    +                   Checkpoints.disposeSavepoint(
    +                           savepointPath,
    +                           jobMasterConfiguration.getConfiguration(),
    +                           userCodeLoader,
    +                           log);
    +           } catch (FlinkException | IOException de) {
    --- End diff --
    
    Why `de`? Just want to make sure it's not a typo.


---

Reply via email to