On Wed, 3 Jun 2026 14:52:49 GMT, Sruthy Jayan <[email protected]> wrote:
>> Modern AIX7+ supports full POSIX TZ format (e.g., >> TZ=CET-1CEST,M3.5.0,M10.5.0), but the current implementation in >> mapPlatformToJavaTimezone() searches the tzmappings file using the complete >> string including DST transition rules. The tzmappings file only contains the >> base timezone identifier (e.g., CET-1CEST), causing the lookup to fail and >> fall back to GMT offset calculation. >> >> This behavior differs from earlier AIX versions and IBM Java 8, which >> truncated the TZ string at the first comma before performing the lookup, >> successfully mapping to proper IANA timezone IDs. >> >> >> Fix include the below : >> Refactor the AIX timezone mapping implementation to support both modern and >> legacy POSIX TZ formats: >> >> Extract mapping logic into new getJavaTimezoneFromPlatform() helper function >> Implement two-phase lookup strategy: >> Phase 1: Attempt to map the full POSIX TZ string (supports modern AIX7+) >> Phase 2: If no match, truncate at first comma and retry (maintains backward >> compatibility) >> Fall back to GMT offset only if both lookups fail >> >> Exclude AIX from CustomTzIDCheckDST.java (validates POSIX DST rules directly) >> Add new AIXTzMappingTest.java to validate AIX-specific behavior: >> Successful mapping: CET-1CEST,M3.5.0,M10.5.0 → Europe/Paris >> Successful mapping: MEZ-1MESZ,M3.5.0,M10.5.0/3 → Europe/Berlin >> GMT fallback: Unknown timezone → GMT+HH:MM >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Sruthy Jayan has updated the pull request incrementally with one additional > commit since the last revision: > > Updated the bug id in the testcase > > Signed-off-by: Sruthy Jayan <[email protected]> test/jdk/java/util/TimeZone/CustomTzIDCheckDST.java line 29: > 27: * @summary This test will ensure that daylight savings rules are followed > 28: * appropriately when setting a custom timezone ID via the TZ env > variable. > 29: * @requires os.family != "windows" & os.family != "aix" Add some comments why AIX is excluded in this test (and refer to the other test) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31270#discussion_r3350637149
