AHeise commented on a change in pull request #11868:
URL: https://github.com/apache/flink/pull/11868#discussion_r443493125
##########
File path:
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/environment/CheckpointConfig.java
##########
@@ -414,9 +417,19 @@ public void enableUnalignedCheckpoints() {
}
/**
- * Returns whether checkpoints should be persisted externally.
+ * Disables unaligned checkpoints.
*
- * @return <code>true</code> if checkpoints should be externalized.
+ * @see #enableUnalignedCheckpoints()
+ */
+ @PublicEvolving
+ public void disableUnalignedCheckpoints() {
Review comment:
Extracted FLINK-18403, which will be backported (while this PR should
only go to master).
##########
File path:
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/graph/StreamingJobGraphGenerator.java
##########
@@ -500,8 +500,9 @@ private void setVertexConfig(Integer vertexID, StreamConfig
config,
config.setStateBackend(streamGraph.getStateBackend());
config.setCheckpointingEnabled(checkpointCfg.isCheckpointingEnabled());
-
config.setUnalignedCheckpointsEnabled(checkpointCfg.isUnalignedCheckpointsEnabled());
config.setCheckpointMode(getCheckpointingMode(checkpointCfg));
+
config.setUnalignedCheckpointsEnabled(config.getCheckpointMode() ==
CheckpointingMode.EXACTLY_ONCE &&
Review comment:
Extracted FLINK-18403, which will be backported (while this PR should
only go to master).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
