| From: | Andrey Borodin <x4mmm(at)yandex-team(dot)ru> |
|---|---|
| To: | Dmitry <dsy(dot)075(at)yandex(dot)ru> |
| Cc: | Álvaro Herrera <alvherre(at)kurilemu(dot)de>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: IPC/MultixactCreation on the Standby server |
| Date: | 2025-07-31 06:29:25 |
| Message-ID: | F5CF7FC1-955E-4E12-8A51-F4172B6977F2@yandex-team.ru |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> On 29 Jul 2025, at 23:15, Andrey Borodin <x4mmm(at)yandex-team(dot)ru> wrote:
>
> I do not understand it yet.
OK, I figured it out. SimpleLruDoesPhysicalPageExist() was reading a physical file and could race with real extension by ExtendMultiXactOffset().
So I used ExtendMultiXactOffset(actual + 1). I hope this does not open a loop for wraparound...
Here's an updated two patches, one for Postgres 17 and one for mater(with a test).
Best regards, Andrey Borodin.
| Attachment | Content-Type | Size |
|---|---|---|
| v9-0001-Avoid-edge-case-2-in-multixacts.patch | application/octet-stream | 15.0 KB |
| v9-PG17-0001-Avoid-edge-case-2-in-multixacts.patch | application/octet-stream | 6.3 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrei Lepikhov | 2025-07-31 07:10:00 | Re: track generic and custom plans in pg_stat_statements |
| Previous Message | Pavel Stehule | 2025-07-31 05:45:28 | Re: proposal: plpgsql, new check for extra_errors - strict_expr_check |