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 reasonable to me. ------------- Marked as reviewed by dfuchs (Reviewer). PR: https://git.openjdk.org/jdk/pull/9293
