You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "MartijnVisser (via GitHub)" <gi...@apache.org> on 2026/09/28 11:07:24 UTC

[PR] [BP-2.2][FLINK-38908][flink-runtime][test] Fix flaky ExecutionPartitionLifecycleTest [flink]

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

   Backport of https://github.com/apache/flink/pull/27406 (76fbab4f20c), which went to master and release-2.3 but not to this branch.
   
   The test uses `forMainThread()` next to a separate I/O executor. The main-thread step of `Execution.deploy()` then runs on the I/O thread and fails its thread assertion, so the partitions are released too early. The fix waits for the deployment descriptor and runs the scheduler calls on a dedicated main-thread executor.
   
   Last occurrence on this branch in the nightly of 2026-09-25, `test_cron_jdk21 core`: https://dev.azure.com/apache-flink/98463496-1af2-4620-8eab-a2ecc1a2e6fe/_build/results?buildId=79415&view=logs&j=d06b80b4-9e88-5d40-12a2-18072cf60528
   
   Verified locally on JDK 17 under CPU load, with assertions enabled as in surefire. `ExecutionPartitionLifecycleTest` fails in 14 of 500 runs on the base commit and in 0 of 500 with this change. 13 of the 14 failures show the CI error. `spotless:check` and `checkstyle:check` pass.
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes (please specify the tool below)
   
   Generated-by: Claude Code (Claude Opus 5.5)
   


-- 
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-unsubscribe@flink.apache.org

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


Re: [PR] [BP-2.2][FLINK-38908][flink-runtime][test] Fix flaky ExecutionPartitionLifecycleTest [flink]

Posted by "MartijnVisser (via GitHub)" <gi...@apache.org>.
MartijnVisser merged PR #29319:
URL: https://github.com/apache/flink/pull/29319


-- 
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-unsubscribe@flink.apache.org

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


Re: [PR] [BP-2.2][FLINK-38908][flink-runtime][test] Fix flaky ExecutionPartitionLifecycleTest [flink]

Posted by "flinkbot (via GitHub)" <gi...@apache.org>.
flinkbot commented on PR #29319:
URL: https://github.com/apache/flink/pull/29319#issuecomment-5868866177

   <!--
   Meta data
   {
     "version" : 1,
     "metaDataEntries" : [ {
       "hash" : "6bda7c25cb1bce0384a3713a9237ba3e7bfd06df",
       "status" : "UNKNOWN",
       "url" : "TBD",
       "triggerID" : "6bda7c25cb1bce0384a3713a9237ba3e7bfd06df",
       "triggerType" : "PUSH"
     } ]
   }-->
   ## CI report:
   
   * 6bda7c25cb1bce0384a3713a9237ba3e7bfd06df UNKNOWN
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run azure` re-run the last Azure build
   </details>


-- 
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-unsubscribe@flink.apache.org

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