On Mon, 7 Apr 2025 09:08:17 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> Volkan Yazici has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add -Djdk.httpclient.keepalive.timeout=120` to `@run` > > test/jdk/java/net/httpclient/HttpResponseConnectionLabelTest.java line 33: > >> 31: * >> 32: * @comment `othervm` usage is intentional since this test uses >> `ForkJoinPool.commonPool()` >> 33: * @comment There is no guarantee that sequential requests will be >> executed immediately one after the other. > > Thank you for all the updates so far Volkan. This is overall looking good to > me. One final nit about this comment. I think we can leave out a lot of this > text and just state something like: > > > @comment use a higher idle timeout to increase the chances of > the same connection being used for sequential HTTP requests Simplified as suggested in db61a01542ce12496721857bb8581683bff2cf81. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24154#discussion_r2030847366