On Tue, 22 Nov 2022 16:24:52 GMT, Michael McMahon <micha...@openjdk.org> wrote:
>> Daniel Fuchs has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Update test/jdk/sun/net/www/http/HttpClient/MultiThreadTest.java >> >> Co-authored-by: Andrey Turbanov <turban...@gmail.com> >> - Update test/jdk/sun/net/www/http/HttpClient/MultiThreadTest.java >> >> Co-authored-by: Andrey Turbanov <turban...@gmail.com> > > test/jdk/sun/net/www/http/HttpClient/MultiThreadTest.java line 83: > >> 81: // The number of time an idle timeout was detected by any worker >> 82: // on the server side. >> 83: static final AtomicInteger IDLE_TIME_EXCEEDED = new AtomicInteger(); > > IDLE_TIME_EXCEEDED is allocated and incremented, but not referenced anywhere > else. Did you mean to include it in a debug log or something? Yes - I meant to use it but in the end it was also necessary to include the time the worker has been idle since it last read/wrote a byte and the end of the test (the time at which all requests had been sent) - so I ended up not using it. I could remove it. ------------- PR: https://git.openjdk.org/jdk/pull/11268
