On Fri, 30 Aug 2024 13:26:57 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
> The fired assert appears to be too strong and can be safely removed. The test > uses a "Staller" to stall the delivery of bytes to the body handlers. > However, the stalling happens for the push promise response bodies, and > there's no guarantee the main response body can't be fully completed before > the push promise bodies start being delivered. > > I have replaced the assert with a comment, and also updated the test to make > it easier to parse logs and diagnose future issues. This pull request has now been integrated. Changeset: b1163bcc Author: Daniel Fuchs <dfu...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/b1163bcc88a5b88b9a56d5584310f1d679690ab2 Stats: 22 lines in 1 file changed: 17 ins; 0 del; 5 mod 8256211: assert fired in java/net/httpclient/DependentPromiseActionsTest (infrequent) Reviewed-by: jpai ------------- PR: https://git.openjdk.org/jdk/pull/20795
