[ 
https://issues.apache.org/jira/browse/FLINK-4510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15455491#comment-15455491
 ] 

ASF GitHub Bot commented on FLINK-4510:
---------------------------------------

Github user StephanEwen commented on the issue:

    https://github.com/apache/flink/pull/2453
  
    Thanks for looking into this.
    
    I wonder if we can do this simpler, without changes to the 
CheckpointCoordinator.
    The only thing that really needs to change is that without periodic 
checkpoints, the `startCheckpointScheduler()` method is not called, which means 
that the JobStatusListener that "CheckpointActivatorDeactivator" should not be 
created and started.



> Always create CheckpointCoordinator
> -----------------------------------
>
>                 Key: FLINK-4510
>                 URL: https://issues.apache.org/jira/browse/FLINK-4510
>             Project: Flink
>          Issue Type: Sub-task
>          Components: State Backends, Checkpointing
>            Reporter: Ufuk Celebi
>            Assignee: Jark Wu
>
> The checkpoint coordinator is only created if a checkpointing interval is 
> configured. This means that no savepoints can be triggered if there is no 
> checkpointing interval specified.
> Instead we should always create it and allow an interval of 0 for disabled 
> periodic checkpoints. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to