Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/3374
Hi @tony810430 thank you for the pull request!
The code looks good.
My feeling is, though, that the number of checkpoints to retain is
something that we want rather in the configuration of the JobManager, than in
the programs snapshot settings.
Think of it like that: There are often two roles, *developer* and *ops*.
- The developer writes the streaming program, and sets the values for the
snapshot settings, like what checkpoint interval would work well for the
application etc.
- The ops person writes the cluster's config and is concerned with
running the job reliably.
Having multiple retained checkpoints is something that concerns more the
ops person.
What do you think?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
