Re: PATCH: 9.5 replication origins fix for logical decoding - Mailing list pgsql-hackers

From Craig Ringer
Subject Re: PATCH: 9.5 replication origins fix for logical decoding
Date
Msg-id CAMsr+YHzU7mnh5cbo6_ZwCR+vpRN7miy=ZqE-Bt8BNaz_Au3Fg@mail.gmail.com
Whole thread Raw
In response to Re: PATCH: 9.5 replication origins fix for logical decoding  (Craig Ringer <craig@2ndquadrant.com>)
Responses Re: PATCH: 9.5 replication origins fix for logical decoding  (Craig Ringer <craig@2ndquadrant.com>)
List pgsql-hackers
On 16 October 2015 at 11:51, Craig Ringer <craig@2ndquadrant.com> wrote:
> Document it as a "don't do that, if you do it you get to keep the pieces"?

Thinking about this some more, having per-change origins makes sense
when you're not using origin LSNs, such as when you're not replaying
from another PostgreSQL instance. So I _can_ see why it exists.

I guess this is mostly a matter of adding some comments and/or some
notes in the functions' docs to explain how it all fits together -
that origins can be per-change, that the txn origin is the origin that
was in effect at commit time, and that the lsn and commit timestamp
are always those that were set at commit time, so you cannot use a
per-change origin with the txn's lsn and expect it to make sense.

Reasonable?

-- Craig Ringer                   http://www.2ndQuadrant.com/PostgreSQL Development, 24x7 Support, Training & Services



pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: Parallel Seq Scan
Next
From: Amit Kapila
Date:
Subject: Re: Parallel Seq Scan