RFR: 8330053: JFR: Use LocalDateTime instead ZonedDateTime [v3]
Markus Grönlund
mgronlun at openjdk.org
Wed Apr 17 13:17:00 UTC 2024
On Wed, 17 Apr 2024 09:48:29 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:
>> Could I have a review of PR that changes so LocalDateTime instead of ZonedDateTime is used when naming chunk files. Problem is that a java.time.DateTimeException occurs when ZonedDateTime.now() is used and the time zone is invalid. This happens in a JCK test.
>>
>> Testing: jdk/jdk/jfr + applicable test in JCK.
>>
>> Thanks
>> Erik
>
> Erik Gahlin has updated the pull request incrementally with one additional commit since the last revision:
>
> Fallback for invalid timezone
Marked as reviewed by mgronlun (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/18729#pullrequestreview-2006014955
More information about the hotspot-jfr-dev
mailing list