Github user tony810430 commented on the issue:
https://github.com/apache/flink/pull/3374
Hi @StephanEwen thanks for your feedback.
I totally agree your opinion. I will make this setting be configured in
`FlinkConfiguration` and be passed to `CheckpointRecoveryFactory`.
Besides, I have some questions for the following implementations.
- Should the verification of this setting throw exception or make it be the
default value when the setting is set with illegal value like `-1`.
- Where should I verify this setting If we need to throw the exception? I
couldn't find the validation util for `FlinkConfiguration`, so I don't know
where I should place my code.
- Should I remain the setting to provide flexibility for _developers_ or
just make it be the _ops_' responsibility only?
Looking forward to having your opinion. Thank you.
---
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.
---
