On Tue, 5 Sep 2023 20:06:41 GMT, Sean Coffey <coff...@openjdk.org> wrote:
> Update the recently added LoggerFinder tests to cater for a possible > condition where the test finishes before the boot logger executor service has > flushed its pending data. > > By simulating a slow thread in the ExecutorService used in BootstrapLogger, I > was able to reproduce the issue described. Adding the > `BootstrapLoggerUtils.awaitPending();` call allows the tests to pass. LGTM! Thanks for the update Seán. ------------- Marked as reviewed by dfuchs (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/15571#pullrequestreview-1617603386