RFR: 8278137: JFR: PrettyWriter uses incorrect year specifier

Aleksey Shipilev shade at openjdk.java.net
Thu Dec 2 11:17:44 UTC 2021


Found by SonarCloud, actually. For this commit: https://github.com/openjdk/jdk/commit/ee3576a48b700df3d8ad4bd447346d4102b20818 -- it says:

> Week Year ("YYYY") should not be used for date formatting
> 
> Few developers are aware of the difference between Y for "Week year" and y for Year when formatting and parsing a date with SimpleDateFormat or DateTimeFormatter. That's likely because for most dates, Week year and Year are the same, so testing at any time other than the first or last week of the year will yield the same value for both y and Y. But in the last week of December and the first week of January, you may get unexpected results.

See also ErrorProne rule: https://errorprone.info/bugpattern/MisusedWeekYear

A nice end-of-the-year time-bomb, that :)

Additional testing:
 - [x] Linux x86_64 fastdebug `jdk_jfr`

-------------

Commit messages:
 - Fix

Changes: https://git.openjdk.java.net/jdk/pull/6665/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6665&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8278137
  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/6665.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/6665/head:pull/6665

PR: https://git.openjdk.java.net/jdk/pull/6665


More information about the hotspot-jfr-dev mailing list