]> granicus.if.org Git - postgresql/commit
Fix timestamptz regression test to still work with latest IANA zone data.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 9 Mar 2017 22:20:11 +0000 (17:20 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 9 Mar 2017 22:20:11 +0000 (17:20 -0500)
commit52f6b2e5a640ab2e6a7a214cad7e95995e3c211d
tree81dfd83e05e6fcd141c91f9c03bc0ca31a799867
parent0ab75448eadec9712716276bb4a09d8e6a70f20c
Fix timestamptz regression test to still work with latest IANA zone data.

The IANA timezone crew continues to chip away at their project of removing
timezone abbreviations that have no real-world currency from their
database.  The tzdata2017a update removes all such abbreviations for
South American zones, as well as much of the Pacific.  This breaks some
test cases in timestamptz.sql that were expecting America/Santiago and
America/Caracas to have non-numeric abbreviations.

The test cases involving America/Santiago seem to have selected that
zone more or less at random, so just replace it with America/New_York,
which is of similar longitude.  The cases involving America/Caracas are
harder since they were chosen to test a time-varying zone abbreviation
around a point where it changed meaning in the backwards direction.
Fortunately, Europe/Moscow has a similar case in 2014, and the MSK/MSD
abbreviations are well enough attested that IANA seems unlikely to
decide to remove them from the database in future.

With these changes, this regression test should pass when using any IANA
zone database from 2015 or later.  One could wish that there were a few
years more daylight on how out-of-date your zone database can be ... but
really the --with-system-tzdata option is only meant for use on platforms
where the zone database is kept up-to-date pretty faithfully, so I do not
think this is a big objection.

Discussion: https://postgr.es/m/6749.1489087470@sss.pgh.pa.us
src/test/regress/expected/timestamptz.out
src/test/regress/sql/timestamptz.sql