On Sun, 1 Feb 2026 08:20:32 GMT, Alan Bateman <[email protected]> wrote:
>> SendaoYan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Use ExecutorService.close() instead of >> ExecutorService.shutdown()/ExecutorService.awaitTermination() > > test/jdk/tools/jimage/VerifyJimage.java line 26: > >> 24: import jdk.internal.jimage.BasicImageReader; >> 25: import jtreg.SkippedException; >> 26: import jdk.test.lib.Utils; > > Using ExecutorService::close makes it simpler, good. > > I assume this import is left over from one of the iterations where you were > adjusting the timeout. Sorry forgot to check it... Fixed. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/29498#discussion_r2750987519
