On Wed, 23 Aug 2023 16:32:32 GMT, Daniel Fuchs <[email protected]> wrote:

>The advantage of using close() is that it is much cleaner and simpler. The 
>disadvantage is that if it fails in timeout there, you have no clue about what 
>prevented close() to finish. In opposition the tracker will provide diagnosis 
>if the client fails to close in the imparted delay.

That sounds reasonable to me. The changes look fine in this PR.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15307#discussion_r1304196554

Reply via email to