spoon-lz commented on code in PR #24461:
URL: https://github.com/apache/flink/pull/24461#discussion_r1560395390


##########
flink-python/docs/reference/pyflink.datastream/checkpoint.rst:
##########
@@ -81,7 +81,7 @@ The default limit of concurrently happening checkpoints: one.
     CheckpointConfig.set_checkpoint_storage
     CheckpointConfig.set_checkpoint_storage_dir
     CheckpointConfig.get_checkpoint_storage
-    ExternalizedCheckpointCleanup
+    ExternalizedCheckpointRetention

Review Comment:
   Is `CheckpointConfig.is_externalized_checkpoints_enabled` or 
`CheckpointConfig.get_externalized_checkpoint_cleanup` ?
   The methods in`checkpoint_config.py` : `set_externalized_checkpoint_cleanup` 
and `get_externalized_checkpoint_cleanup` both seem to need to be renamed
   



-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to