Zakelly
Zakelly
@spoon-lz Sorry for the late reply, seems we come to an agreement in discussion. To be brief, we agree to deprecate the old class and introduce a new one. Corresponding...
> @Zakelly I also saw the conclusion of the discussion in the email. I will re-edit this PR after #24381 is merged. Another question is, "Split ExternalizedCheckpointCleanup out of CheckpointConfig...
> @Zakelly There are some references to `org.apache.flink.streaming.api.CheckpointingMode` in `ExecutionCheckpointingOptions`. When we move these configurations into `CheckpointingOptions`, do we need to replace them with `org.apache.flink .core.execution.CheckpointingMode`? @spoon-lz Firstly, we should...
@spoon-lz I didn't see any new commit on https://github.com/spoon-lz/flink/tree/lz-CheckpointingOptions
I'll merge this once the CI green.
@flinkbot run azure
@masteryhx I have transformed to the new CheckpointMode everywhere, and rebased master. Would you please take a look?
Rebased for conflicts....
@flinkbot run azure
@flinkbot run azure