Re: tracking commit timestamps - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: tracking commit timestamps
Date
Msg-id CAB7nPqS+T+-y2t-p88pwO3DG5-QJ9L_9k0yq5qt01ia0QCJc9A@mail.gmail.com
Whole thread Raw
In response to Re: tracking commit timestamps  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Responses Re: tracking commit timestamps  (Andres Freund <andres@2ndquadrant.com>)
List pgsql-hackers
On Tue, Nov 4, 2014 at 10:01 PM, Alvaro Herrera <alvherre@2ndquadrant.com> wrote:
Michael Paquier wrote:

> I'm still on a -1 for that. You mentioned that there is perhaps no reason
> to delay a decision on this matter, but IMO there is no reason to rush
> either in doing something we may regret. And I am not the only one on this
> thread expressing concern about this extra data thingy.
>
> If this extra data field is going to be used to identify from which node a
> commit comes from, then it is another feature than what is written on the
> subject of this thread. In this case let's discuss it in the thread
> dedicated to replication identifiers, or come up with an extra patch once
> the feature for commit timestamps is done.

Introducing the extra data field in a later patch would mean an on-disk
representation change, i.e. pg_upgrade trouble.
Then why especially 4 bytes for the extra field? Why not 8 or 16?
--
Michael

pgsql-hackers by date:

Previous
From: Marko Tiikkaja
Date:
Subject: to_char_at_timezone()?
Next
From: Tom Lane
Date:
Subject: Re: to_char_at_timezone()?