On Fri, 8 Dec 2023 06:56:03 GMT, Jaikiran Pai <[email protected]> wrote:

> Can I please get a review of this trivial test only change which adds the 
> `-Djdk.tracePinnedThreads=full` system property when launching the 
> `HttpClientLocalAddrTest` test?
> 
> This test has failed with timeout (on a rare occasion). The test uses virtual 
> threads for the HttpClient instances it uses and has 2 `@run` test 
> definitions. One of it already uses `-Djdk.tracePinnedThreads=full` to help 
> debug any pinned thread issues. The commit in this PR updates the other 
> `@run` definition to use this same system property for the same reason.
> 
> The test continues to pass with this change.

This pull request has now been integrated.

Changeset: f577385f
Author:    Jaikiran Pai <[email protected]>
URL:       
https://git.openjdk.org/jdk/commit/f577385fc8d5a6f4c47d88e8f9166a7b76d1246e
Stats:     1 line in 1 file changed: 1 ins; 0 del; 0 mod

8316738: java/net/httpclient/HttpClientLocalAddrTest.java failed in timeout

Reviewed-by: dfuchs

-------------

PR: https://git.openjdk.org/jdk/pull/17031

Reply via email to