Yun Tang created FLINK-13281:
--------------------------------

             Summary: Fix the verify logic of 
AdaptedRestartPipelinedRegionStrategyNGAbortPendingCheckpointsTest
                 Key: FLINK-13281
                 URL: https://issues.apache.org/jira/browse/FLINK-13281
             Project: Flink
          Issue Type: Bug
          Components: Tests
    Affects Versions: 1.9.0
            Reporter: Yun Tang
            Assignee: Yun Tang


FLINK-13060 ensure FailoverStrategies should respect restart constraints. 
However, this change did not cover the 
{{AdaptedRestartPipelinedRegionStrategyNGAbortPendingCheckpointsTest}} which 
should also set as {{FixedDelayRestartStrategy}}. Currently, as the pending 
checkpoint would still be discarded due to 
[failUnacknowledgedPendingCheckpoints|https://github.com/apache/flink/blob/a7bdbb9aff734fc586e74c95984c7f956dbf576c/flink-runtime/src/main/java/org/apache/flink/runtime/checkpoint/CheckpointCoordinator.java#L977]
 within {{CheckpointCoordinator}}, 
{{AdaptedRestartPipelinedRegionStrategyNGAbortPendingCheckpointsTest}} could 
pass the test.

This is actually not what we want the test did, we should actually verify that 
when region failover, pending checkpoints would be discarded.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to