pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Andrew Gierth <andrew@tao11.riddles.org.uk>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: Stephen Frost <sfrost@snowman.net>
Cc: Andres Freund <andres@anarazel.de>
Cc: Christoph Berg <myon@debian.org>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Subject: Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)
Date: Thu, 20 Jun 2019 00:59:29 +0100
Message-ID: <877e9h18df.fsf@news-spur.riddles.org.uk> (raw)
In-Reply-To: <27991.1560984458@sss.pgh.pa.us>
References: <24452.1560285699@sss.pgh.pa.us>
	<20190614095546.GA15107@msg.df7cb.de>
	<26948.1560517875@sss.pgh.pa.us>
	<87ef3w3u8a.fsf@news-spur.riddles.org.uk>
	<7114.1560539536@sss.pgh.pa.us>
	<875zp74zpf.fsf@news-spur.riddles.org.uk>
	<871rzv4yes.fsf@news-spur.riddles.org.uk>
	<20190617173943.z3ttxqenbcwrlag4@alap3.anarazel.de>
	<20190617183458.GG2480@tamriel.snowman.net>
	<20190617183828.hj7ci362zbxd4nax@alap3.anarazel.de>
	<20190617184053.GI2480@tamriel.snowman.net>
	<CA+TgmoYF00U3Pbkh4mXhXr1NN6pKFM278KmXRS35HDuHgNcMFw@mail.gmail.com>
	<25314.1560980148@sss.pgh.pa.us>
	<27991.1560984458@sss.pgh.pa.us>

>>>>> "Tom" == Tom Lane <tgl@sss.pgh.pa.us> writes:

 Tom> So I'm toying with the idea of extending Andrew's patch to put a
 Tom> negative preference on "localtime", ensuring we'll use some other
 Tom> name for the zone if one is available.

 Tom> Also, now that we have this mechanism, maybe we should charge it
 Tom> with de-preferencing the old "Factory" zone, removing the
 Tom> hard-wired kluge that we currently have for rejecting that.
 Tom> (Modern tzdb doesn't install "Factory" at all, but some
 Tom> installations might still do so in the service of blind backwards
 Tom> compatibility.)

I was planning on submitting a follow-up myself (for pg13+) for
discussion of further improvements. My suggestion would be that we
should have the following order of preference, from highest to lowest:

 - UTC  (justified by being an international standard)
 
 - Etc/UTC

 - zones in zone.tab/zone1970.tab:

     These are the zone names that are intended to be presented to the
     user to select from. Dispute the exact meaning as you will, but I
     think it makes sense that these names should be chosen over
     equivalently good matches just on that basis.

 - zones in Africa/ America/ Antarctica/ Asia/ Atlantic/ Australia/
   Europe/ Indian/ Pacific/ Arctic/

     These subdirs are the ones generated by the "primary" zone data
     files, including both Zone and Link statements but not counting
     the "backward" and "etcetera" files.

 - GMT  (justified on the basis of its presence as a default in the code)

 - Etc/*

 - any other zone name with a /

 - any zone name without a /, excluding 'localtime' and 'Factory'

 - 'localtime'

 - 'Factory'

Choosing names with / over ones without is a change from our existing
preference for shorter names, but it's more robust in the face of the
various crap that gets dumped in the top level of the zoneinfo dir.
It could be argued that we should reverse the relative order of UTC vs.
Etc/UTC and likewise for GMT for the same reason, but I think that's
less important.

-- 
Andrew (irc:RhodiumToad)





view thread (74+ messages)  latest in thread

Message-ID: <877e9h18df.fsf@news-spur.riddles.org.uk>
Permalink:  ../877e9h18df.fsf@news-spur.riddles.org.uk/
Also on:    postgresql.org/message-id/877e9h18df.fsf@news-spur.riddles.org.uk

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: andrew@tao11.riddles.org.uk, tgl@sss.pgh.pa.us, robertmhaas@gmail.com, sfrost@snowman.net, andres@anarazel.de, myon@debian.org, pgsql-hackers@postgresql.org
  Subject: Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)
  In-Reply-To: <877e9h18df.fsf@news-spur.riddles.org.uk>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox