You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2021/08/11 00:16:21 UTC

[GitHub] [beam] ibzib commented on a change in pull request #15290: [BEAM-10955] Update Flink minor versions and enable testSavepointRest…

ibzib commented on a change in pull request #15290:
URL: https://github.com/apache/beam/pull/15290#discussion_r686403257



##########
File path: runners/flink/src/test/java/org/apache/beam/runners/flink/FlinkSavepointTest.java
##########
@@ -142,7 +141,6 @@ public void afterTest() throws Exception {
     ensureNoJobRunning();
   }
 
-  @Ignore("https://issues.apache.org/jira/projects/BEAM/issues/BEAM-10955")

Review comment:
       According to the jira https://issues.apache.org/jira/browse/FLINK-23201, the underlying issue was only patched for Flink 1.12-1.14, since Flink only maintains the newest three minor versions at a time. Since it's not patched in Flink 1.11, we should either fork this test file so this test doesn't run on 1.11, or more simply we could just wait until dropping Flink 1.11 support in Beam before re-enabling the test.




-- 
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: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org