pnowojski commented on a change in pull request #11868:
URL: https://github.com/apache/flink/pull/11868#discussion_r442938829
##########
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:
> would be relevant when UC becomes the default
That's a point, but maybe more in favour of dropping
`enableUnalignedCheckpoints ()`? What about keeping and using just
`setUnalignedCheckpoints(boolean)` (without the `Enable` infix)?
----------------------------------------------------------------
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]
