From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andres Freund <andres@anarazel.de>
Cc: Jim Nasby <Jim.Nasby@BlueTreble.com>
Cc: Petr Jelinek <petr.jelinek@2ndquadrant.com>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: pgsql-hackers@postgresql.org <pgsql-hackers@postgresql.org>
Subject: Re: Replication vs. float timestamps is a disaster
Date: Wed, 22 Feb 2017 17:45:52 -0500
Message-ID: <21225.1487803552@sss.pgh.pa.us> (raw)
In-Reply-To: <20170222135634.gnlqu6by2xbpvwii@alap3.anarazel.de>
References: <26788.1487455319@sss.pgh.pa.us>
<CA+TgmoafyUzmg8T-rKEgmkX7F1iQLSrKBmqKa26EattXpMS-ng@mail.gmail.com>
<1691.1487519369@sss.pgh.pa.us>
<20170220070311.gscn7ppqbwlgdpjq@alap3.anarazel.de>
<3bd3c551-8b23-45c8-27a2-97b5a922b130@2ndquadrant.com>
<20170220110456.2jupmfhxlizykmmc@alap3.anarazel.de>
<fece0003-6ede-ca8b-8f26-3c6fa899583d@BlueTreble.com>
<20170222075345.mbzrmdaetdqfgitk@alap3.anarazel.de>
<21041.1487771008@sss.pgh.pa.us>
<20170222135634.gnlqu6by2xbpvwii@alap3.anarazel.de>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>
Andres Freund <andres@anarazel.de> writes:
> On 2017-02-22 08:43:28 -0500, Tom Lane wrote:
>> (To be concrete, I'm suggesting dropping --disable-integer-datetimes
>> in HEAD, and just agreeing that in the back branches, use of replication
>> protocol with float-timestamp servers is not supported and we're not
>> going to bother looking for related bugs there. Given the lack of field
>> complaints, I do not believe anyone cares.)
> I think we should just fix it in the back branches, and drop the float
> timestamp code in 10 or 11. I.e. 1) and then 4).
I'm not personally interested in touching this issue in the back branches,
but if you want to spend time on it, I surely won't stand in your way.
What I *am* willing to spend time on is removing float-timestamp code
in HEAD. I've not yet heard anybody speak against doing that (or at
least, nothing I interpreted as a vote against it). If I've not heard
any complaints by tomorrow, I'll get started on that.
I envision the following work plan:
* Reject --disable-integer-datetimes in configure. To avoid breaking
existing build scripts unnecessarily, --enable-integer-datetimes will
still be accepted.
* Convert HAVE_INT64_TIMESTAMP to a fixed, always-defined symbol.
(We shouldn't remove it entirely because that would break third-party
code that might be testing it. Perhaps shove it in pg_config_manual.h.)
* Possibly remove the enableIntTimes field from pg_control as well
as associated logic in places like pg_controldata. There might be some
argument for keeping this, though ... anyone have an opinion? pg_upgrade,
at least, would need a bespoke test instead.
* Remove appropriate user documentation.
* Remove all core-code tests for HAVE_INT64_TIMESTAMP, along with the
negative sides of those #if tests.
* Change the places in the replication code that declare timestamp
variables to declare them as TimestampTz not int64, and adjust nearby
comments accordingly. (This will just be code beautification at this
point, but it still seems like a good idea.)
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
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-hackers@postgresql.org
Cc: tgl@sss.pgh.pa.us, andres@anarazel.de, Jim.Nasby@BlueTreble.com, petr.jelinek@2ndquadrant.com, robertmhaas@gmail.com
Subject: Re: Replication vs. float timestamps is a disaster
In-Reply-To: <21225.1487803552@sss.pgh.pa.us>
* 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