tkaymak opened a new pull request, #38273:
URL: https://github.com/apache/beam/pull/38273

   ## Summary
   
   The four portable-execution test methods in `FlinkRequiresStableInputTest` 
were declared with `@Test(timeout = 30_000)`. That budget is consistently 
exceeded under CI load and currently causes 
`testParDoRequiresStableInputPortable` to fail with `TestTimedOutException at 
FlinkRequiresStableInputTest.java:190` (the 1-second poll loop in 
`executePipeline`) on **every recent `beam_PreCommit_Java` run on master**.
   
   The four sibling portable-pipeline tests in the same package already use 
`@Test(timeout = 120_000)`:
   
   - `PortableExecutionTest`
   - `PortableTimersExecutionTest`
   - `PortableStateExecutionTest`
   - `ReadSourcePortableTest`
   
   `FlinkRequiresStableInputTest` is doing strictly more work than those 
(deliberate failure → checkpoint → restore → re-execute → DONE) yet had a 4× 
tighter budget, which matches the "Jenkins under high load" diagnosis on #21333.
   
   This PR aligns the timeout with the rest of the portable Flink test suite. 
No production code touched, no test semantics changed.
   
   Addresses #21333 (the Flink-1.x portable path; the Flink 2.0 override 
remains `@Ignore`'d per the separate DataStream-only-batch issue called out in 
#21333 on 2025-12-23).
   
   ## Test plan
   
   - [ ] `beam_PreCommit_Java` green on this PR
   - [ ] CI on master starts going green again once merged
   
   ------------------------
   
   Thank you for your contribution! Follow this checklist to help us 
incorporate your contribution quickly and easily:
   
    - [x] Mention the appropriate issue in your description (for example: 
`addresses #123`), if applicable. This will automatically add a link to the 
pull request in the issue. If you would like the issue to automatically close 
on merging the pull request, comment `fixes #<ISSUE NUMBER>` instead.
    - [ ] Update `CHANGES.md` with noteworthy changes — N/A, test-only.
    - [ ] If this contribution is large, please file an Apache [Individual 
Contributor License Agreement](https://www.apache.org/licenses/icla.pdf) — N/A, 
4-line test fix.


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