On Mon, 27 Jun 2022 11:52:37 GMT, Thejasvi Voniadka <tvonia...@openjdk.org> 
wrote:

>> This is a simple change to add some protective code to 
>> java/util/zip/ZipFile/DeleteTempJar.java so that the clean-up is more 
>> complete in the event of encountering an any exceptions while running. In 
>> the present state, the server.stop() does not get executed in case of 
>> encountering a failure. As a result, the test may get hung until a timeout. 
>> This fix should address the scenario.
>> 
>> The test passes on all supported platforms post fix.
>
> Thejasvi Voniadka has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   8282036: [Test]: Update java/util/zip/ZipFile/DeleteTempJar.java to stop 
> HttpServer cleanly in case of exceptions

Looks okay. You'll ned to edit the edit the title to align with the description 
of the JBS issue before you can integrate.

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

Marked as reviewed by alanb (Reviewer).

PR: https://git.openjdk.org/jdk/pull/9293

Reply via email to