From: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
---|---|
To: | Craig Ringer <craig(at)2ndquadrant(dot)com>, Petr Jelinek <petr(dot)jelinek(at)2ndquadrant(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] Fix minor race in commit_ts SLRU truncation vs lookups |
Date: | 2017-01-12 14:25:15 |
Message-ID: | eb6a149d-e145-a8c5-77fd-20366e7dbf0a@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 12/29/16 4:28 AM, Craig Ringer wrote:
> On 29 December 2016 at 16:51, Craig Ringer <craig(at)2ndquadrant(dot)com> wrote:
>> On 28 December 2016 at 22:16, Petr Jelinek <petr(dot)jelinek(at)2ndquadrant(dot)com> wrote:
>>
>>> About the patch, it looks good to me for master with the minor exception
>>> that:
>>>> + appendStringInfo(buf, "pageno %d, xid %u",
>>>> + trunc.pageno, trunc.oldestXid);
>>>
>>> This should probably say oldestXid instead of xid in the text description.
>>
>> Agreed.
>
> Slightly amended attached.
I've looked over this. It looks correct to me in principle.
The commit message does not actually explain how the race on the standby
is fixed by the patch.
Also, I wonder whether we should not in vacuum.c change the order of the
calls of SetTransactionIdLimit() and SetMultiXactIdLimit() as well, just
to keep everything consistent.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2017-01-12 14:26:42 | Re: parallelize queries containing subplans |
Previous Message | Robert Haas | 2017-01-12 14:10:17 | Re: Obsolete reference to ParallelMain() in a comment |