From: | Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> |
---|---|
To: | er(at)xs4all(dot)nl |
Cc: | boekewurm+postgres(at)gmail(dot)com, pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: TRAP: FailedAssertion("HaveRegisteredOrActiveSnapshot()", File: "toast_internals.c", Line: 670, PID: 19403) |
Date: | 2022-04-01 08:28:52 |
Message-ID: | 20220401.172852.1154281926306406898.horikyota.ntt@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
At Wed, 30 Mar 2022 17:58:24 +0900 (JST), Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> wrote in
> At Wed, 30 Mar 2022 11:46:13 +0900 (JST), Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> wrote in
> > But, in the first place the *fix* has been found to be wrong. I'm
> > going to search for the right fix..
>
> FETCH uses the snapshot at DECLARE. So anyhow I needed to set the
> queryDesk's snapshot used in PortalRunSelect to the FETCH's portal's
> holdSnapshot. What I did in this version is:
By the way, this is, given that the added check in init_toast_snapshot
is valid, a long-standing "bug", which at least back to 12.
I'm not sure what to do for this.
1. ignore the check for certain cases?
2. apply any fix only to master and call it a day. 14 and earlier
doesn't have the assertion check so they don't complain.
3. apply a fix to all affected versions.
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Langote | 2022-04-01 08:36:49 | Re: generic plans and "initial" pruning |
Previous Message | Peter Smith | 2022-04-01 08:22:20 | Re: Tablesync early exit |