On Thu, 21 Sep 2023 18:51:50 GMT, Naoto Sato <na...@openjdk.org> wrote:
>> Fixing a test case that fails in some time zones. Making sure the test is >> run in `UTC` zone will fix the issue. Confirmed the fix by manually setting >> machine's time zone to Europe/Dublin. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > Use createTestJvm for vm invocation The difference between `createJavaProcessBuilder()` and `createTestJvm()` appears to be that additionally `test.vm.opts` and `test.java.opts` system properties get prefixed to the opts that are passed for the process creation. That looks fine to me, for this test. The current state of this PR in `bd786b92` looks good to me. Thank you Naoto for the updates. ------------- Marked as reviewed by jpai (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/15829#pullrequestreview-1639276171