forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitf077e1b
committed
Fix timestamptz regression test to still work with latest IANA zone data.
The IANA timezone crew continues to chip away at their project of removingtimezone abbreviations that have no real-world currency from theirdatabase. The tzdata2017a update removes all such abbreviations forSouth American zones, as well as much of the Pacific. This breaks sometest cases in timestamptz.sql that were expecting America/Santiago andAmerica/Caracas to have non-numeric abbreviations.The test cases involving America/Santiago seem to have selected thatzone more or less at random, so just replace it with America/New_York,which is of similar longitude. The cases involving America/Caracas areharder since they were chosen to test a time-varying zone abbreviationaround a point where it changed meaning in the backwards direction.Fortunately, Europe/Moscow has a similar case in 2014, and the MSK/MSDabbreviations are well enough attested that IANA seems unlikely todecide to remove them from the database in future.With these changes, this regression test should pass when using any IANAzone database from 2015 or later. One could wish that there were a fewyears more daylight on how out-of-date your zone database can be ... butreally the --with-system-tzdata option is only meant for use on platformswhere the zone database is kept up-to-date pretty faithfully, so I do notthink this is a big objection.Discussion:https://postgr.es/m/6749.1489087470@sss.pgh.pa.us1 parentc08d82f commitf077e1b
File tree
2 files changed
+151
-282
lines changed- src/test/regress
- expected
- sql
2 files changed
+151
-282
lines changed0 commit comments
Comments
(0)