Dennis-Mircea commented on code in PR #1205:
URL: 
https://github.com/apache/flink-kubernetes-operator/pull/1205#discussion_r4002868693


##########
examples/basic-checkpoint-ha.yaml:
##########
@@ -31,6 +31,11 @@ spec:
     high-availability:
       type: kubernetes
       storageDir: file:///flink-data/ha
+    # Keep the below property uncommented when running against 
StateMachineExample job on
+    # Flink 2.0 to 2.3 because that job enables async state which does not 
support
+    # canonical savepoints. This is fixed by FLINK-39061 in Flink 2.4 and 
later, so the
+    # property can be removed when running against that job on Flink 2.4 and 
later.
+    # This problem is not in place for Flink 1.20 or lower.

Review Comment:
   I put this comment in those places just to motivate the reason why the 
NATIVE value should stay in place for a certain Flink version. If you think 
it's still unnecessary, I'll remove it, no problem.



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