Zakelly commented on code in PR #24381:
URL: https://github.com/apache/flink/pull/24381#discussion_r1529761768
##########
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/environment/CheckpointConfig.java:
##########
@@ -177,20 +177,44 @@ public boolean isCheckpointingEnabled() {
* Gets the checkpointing mode (exactly-once vs. at-least-once).
*
* @return The checkpointing mode.
+ * @deprecated Use {@link #getConsistencyMode} instead.
*/
- public CheckpointingMode getCheckpointingMode() {
+ @Deprecated
+ public org.apache.flink.streaming.api.CheckpointingMode
getCheckpointingMode() {
return
configuration.get(ExecutionCheckpointingOptions.CHECKPOINTING_MODE);
}
/**
* Sets the checkpointing mode (exactly-once vs. at-least-once).
*
* @param checkpointingMode The checkpointing mode.
+ * @deprecated Use {@link #setConsistencyMode} instead.
*/
- public void setCheckpointingMode(CheckpointingMode checkpointingMode) {
+ @Deprecated
+ public void setCheckpointingMode(
+ org.apache.flink.streaming.api.CheckpointingMode
checkpointingMode) {
configuration.set(ExecutionCheckpointingOptions.CHECKPOINTING_MODE,
checkpointingMode);
}
+ /**
+ * Gets the checkpointing consistency mode (exactly-once vs.
at-least-once).
+ *
+ * @return The checkpointing mode.
+ */
+ public CheckpointingMode getConsistencyMode() {
Review Comment:
@masteryhx Personally, I'd be fine with name the enum
`CheckpointingConsistencyMode`, but that's a breaking change and needs more
discussion. My current plan is to keep the `CheckpointingMode` here, and
provide API with 'Consistency' in its name just for avoiding conflicts. We will
reuse the original API name later in 2.0, WDYT?
--
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
