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. Thank you Daniel for the review. Given that this is a trivial test-only change and tier2 tests pass with this change, I'll go ahead and integrate this now. ------------- PR Comment: https://git.openjdk.org/jdk/pull/17031#issuecomment-1846915996
