[jdk17u] RFR: 8292579: (tz) Update Timezone Data to 2022c [v2]

Andrew John Hughes andrew at openjdk.org
Wed Sep 21 14:20:49 UTC 2022


On Wed, 21 Sep 2022 08:42:49 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

> > Thanks for the backport, looks good to me. Our nightly tests in 17u are green now after your latest adjustment. One additional remark - shouldn't the Kiev -> Kyiv name change also be reflected in the TimeZoneNames*.java files ? I think this was done in previous tzdata updates ; but it is not in the jdk change as well so if needed it might be something for an additional future change.
> 
> Isn't this handled by https://bugs.openjdk.org/browse/JDK-8293834?

Yes, we discussed this on the original PR: https://github.com/openjdk/jdk/pull/10012#issuecomment-1247209990

The fact that this is linked to an existing timezone (Europe/Kiev) seems to remove the need for a new entry, as would be the case if it was a completely new timezone. Previous updates have been to introduce new timezones or change the translations of existing ones; see https://github.com/openjdk/jdk/commit/ad3be04d2ac84836e393d696ff03ddfe72779094 as an example. But, generally, this is something to keep an eye on and I will check it for each update repository once JDK-8292579 is in.

I'll be bringing JDK-8293834 back to the other update releases. It has been reviewed for 19 but still not yet approved. If that doesn't happen today, then I'll propose it for 17 as if it were a new patch, so this doesn't miss rampdown.

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

PR: https://git.openjdk.org/jdk17u/pull/352


More information about the jdk-updates-dev mailing list