From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | "Drouvot, Bertrand" <bdrouvot(at)amazon(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Geoghegan <pg(at)bowt(dot)ie>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: prion failed with ERROR: missing chunk number 0 for toast value 14334 in pg_toast_2619 |
Date: | 2021-07-06 19:06:13 |
Message-ID: | 20210706190612.GM22043@telsasoft.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Jun 30, 2021 at 03:29:41PM +0900, Michael Paquier wrote:
> On Tue, May 18, 2021 at 01:04:18PM +0200, Drouvot, Bertrand wrote:
> > On 5/17/21 8:56 PM, Andres Freund wrote:
> >> On 2021-05-17 20:14:40 +0200, Drouvot, Bertrand wrote:
> >>> I was also wondering if:
> >>>
> >>> * We should keep the old behavior in case pg_resetwal -x is being used
> >>> without -u?
(The proposed patch does not set an arbitrary oldestXID
> >>> anymore in
case -x is used)
> >> I don't think we should. I don't see anything in the old behaviour worth
> >> maintaining.
>
> So, pg_resetwal logic with the oldest XID assignment is causing some
> problem here. This open item is opened for some time now and it is
> idle for a couple of weeks. It looks that we have some solution
> drafted, to be able to move forward, with the following things (no
> patches yet):
> - More robustness safety checks in procarray.c.
> - A rework of oldestXid in pg_resetwal.
>
> Is there somebody working on that?
Bertrand sent a patch which is awaiting review.
This allows both of Tom's reproducers to pass pg_upgrade (with assertions and
delays).
http://cfbot.cputube.org/bertrand-drouvot.html
I re-arranged the pg_upgrade output of that patch: it was in the middle of the
two halves: "Setting next transaction ID and epoch for new cluster"
--
Justin
Attachment | Content-Type | Size |
---|---|---|
0001-prion-failed-with-ERROR-missing-chunk-number-0-for-t.patch | text/x-diff | 2.5 KB |
0002-pg_upgrade-can-result-in-early-wraparound-on-databas.patch | text/x-diff | 10.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Gustafsson | 2021-07-06 19:08:53 | Re: proposal - log_full_scan |
Previous Message | Tom Lane | 2021-07-06 18:57:00 | Re: visibility map corruption |