yanghua commented on a change in pull request #8410: [FLINK-11159] Allow 
configuration whether to fall back to savepoints for restore
URL: https://github.com/apache/flink/pull/8410#discussion_r282921234
 
 

 ##########
 File path: 
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/environment/CheckpointConfig.java
 ##########
 @@ -72,6 +72,9 @@
        /** Determines if a tasks are failed or not if there is an error in 
their checkpointing. Default: true */
        private boolean failOnCheckpointingErrors = true;
 
+       /** Determines if a job will fallback to checkpoint when there is a 
savepoint later than checkpoint. **/
+       private boolean perferCheckpointForRecovery = false;
 
 Review comment:
   I think this config option belongs to checkpoint field. So provide it in 
`CheckpointConfig` sounds more reasonable. What's more, it is a job level 
config option, right? However, `flink-conf.yaml` is a cluster-level 
configuration file(if users deploy Flink with standalone mode, it will affect 
all the jobs).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to