On Thu, 13 Jan 2022 15:39:28 GMT, Julia Boes <jb...@openjdk.org> wrote:

>> This new test confirms that the jwebserver does not wait indefinitely for a 
>> request to arrive, but instead closes the connection when the maximum 
>> request time is reached. To facilitate this, Exchange::run is amended to 
>> process cancelled keys in the case where the current thread is the 
>> dispatcher thread (which is the case for the jwebserver).
>> 
>> Testing: tier 1-3 all clear.
>
> Julia Boes has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   address PR comments:
>   * remove redundant null check
>   * use try-finally to stop process in test

LGTM. @Michael-Mc-Mahon is that the fix you had in mind?

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

Marked as reviewed by dfuchs (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/7052

Reply via email to