gyfora commented on code in PR #25896:
URL: https://github.com/apache/flink/pull/25896#discussion_r1975433457


##########
flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/operator/CEPMigrationTest.java:
##########
@@ -53,9 +54,14 @@
  * Tests for checking whether CEP operator can restore from snapshots that 
were done using previous
  * Flink versions.
  *
+ * <p>NOTE: Due to major upgrades of serializers, Flink 2.0 is not compatible 
with saved state from
+ * Flink 1.x, and this test is temporarily disabled. This test should be 
reinstated for future
+ * versions of Flink, such as Flink 2.1, that promise compatibility with older 
releases.
+ *
  * <p>For regenerating the binary snapshot file of previous versions you have 
to run the {@code
  * write*()} method on the corresponding Flink release-* branch.
  */
+@Ignore("Flink 2.0 is not compatible with saved state from Flink 1.x")

Review Comment:
   looks good! 



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

Reply via email to