DanielLeens commented on PR #12313: URL: https://github.com/apache/seatunnel/pull/12313#issuecomment-6594368162
Closing this PR as superseded by #12316 (merged 2026-10-28). #12316 fixed the engine-side root cause of the `BackpressureSlowSinkIT` instability (the source reader could starve barrier injection on the checkpoint lock; it now hands the lock to waiting barrier injectors). It also rewrote `BackpressureSlowSinkIT` and intentionally kept the original sustained workload: `stream_fast_fakesource_to_slow_inmemory_backpressure.conf` on dev still uses `row.num = 10000000` and `split.num = 1`, so the long-poll case stays covered as the regression test for that fix. This PR switches the fixture to 500 splits of 4000 rows, which avoids long polls and would remove that regression coverage (the same concern raised by goutamadwant and SEZ9 in review). With the root cause fixed on dev there is nothing left for this PR to do. If the IT flakes again on dev, a new issue with the failing run is the right place to track it. -- 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]
