Integrated: 8256211: assert fired in java/net/httpclient/DependentPromiseActionsTest (infrequent)

Daniel Fuchs dfuchs at openjdk.org
Mon Sep 2 14:54:22 UTC 2024


On Fri, 30 Aug 2024 13:26:57 GMT, Daniel Fuchs <dfuchs at 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 <dfuchs at 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


More information about the net-dev mailing list