JingGe commented on code in PR #23758:
URL: https://github.com/apache/flink/pull/23758#discussion_r1399378965


##########
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/environment/CheckpointConfig.java:
##########
@@ -759,9 +765,15 @@ public void setCheckpointStorage(CheckpointStorage 
storage) {
      * Configures the application to write out checkpoint snapshots to the 
configured directory. See
      * {@link FileSystemCheckpointStorage} for more details on checkpointing 
to a file system.
      *
+     * @deprecated The method is marked as deprecated because starting from 
Flink 1.19, the usage of
+     *     all complex Java objects related to configuration, including their 
getter and setter
+     *     methods, should be replaced by ConfigOption. In a future major 
version of Flink, this

Review Comment:
   There are many methods in CheckpointConfig. When should we use ConfigOption 
and when should we use CheckpointConfig directly?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to