amirdeljouyi opened a new pull request, #28591:
URL: https://github.com/apache/flink/pull/28591

   # test: add CheckpointCoordinator scheduling regression tests
   
   ## What is the purpose of the change
   
   This PR adds focused regression coverage for `CheckpointCoordinator` 
scheduling.
   
   Impact on coverage:
   
   Covered lines include 
[`CheckpointCoordinator.java:469-484`](https://github.com/apache/flink/blob/master/flink-runtime/src/main/java/org/apache/flink/runtime/checkpoint/CheckpointCoordinator.java#L469-L484),
 and 
[`CheckpointCoordinator.java:2035-2055`](https://github.com/apache/flink/blob/master/flink-runtime/src/main/java/org/apache/flink/runtime/checkpoint/CheckpointCoordinator.java#L2035-L2055).
   
   Branch coverage for `CheckpointCoordinator` increases by about 4%.
   
   Why:
   
   The tests cover delay normalization when the configured minimum pause is 
very large, and the case where clearing backlog should keep an already earlier 
scheduled checkpoint.
   
   ## Brief change log
   
   - Add regression coverage for scheduler delay normalization.
   - Add regression coverage for backlog rescheduling.
   
   ## Verifying this change
   
   This change added tests and can be verified as follows:
   
   - Run `./mvnw -pl flink-runtime -Dtest=CheckpointCoordinatorTest test`.
   
   ## Does this pull request potentially affect one of the following parts:
   
   - Dependencies (does it add or upgrade a dependency): no
   - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: no
   - The serializers: no
   - The runtime per-record code paths (performance sensitive): no
   - Anything that affects deployment or recovery: Checkpointing tests only; no 
production change
   - The S3 file system connector: no
   
   ## Documentation
   
   - Does this pull request introduce a new feature? no
   - If yes, how is the feature documented? not applicable
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [x] Yes (please specify the tool below)
     Codex 5.5 has been used in making these test cases.


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