I don't think it's appropriate to add 8159684 to TimeZoneTest.java which doesn't test the data changes of 2016e/f at all. I think there should be a time zone data test in java.time to confirm the tzdata changes.

Otherwise, the changes look good to me.

Thanks,
Masayoshi

On 7/12/2016 6:27 PM, Ramanand Patil wrote:
Hi all,

Please review the latest TZDATA integration (tzdata2016f) to JDK9.
Bug: https://bugs.openjdk.java.net/browse/JDK-8159684
Webrev: http://cr.openjdk.java.net/~rpatil/8159684/webrev.00/

All the TimeZone related tests are passed after this integration.

Regards,
Ramanand.

Reply via email to