Re: TRAP: FailedAssertion("HaveRegisteredOrActiveSnapshot()", File: "toast_internals.c", Line: 670, PID: 19403)

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andres Freund <andres(at)anarazel(dot)de>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, Erik Rijkers <er(at)xs4all(dot)nl>, Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: TRAP: FailedAssertion("HaveRegisteredOrActiveSnapshot()", File: "toast_internals.c", Line: 670, PID: 19403)
Date: 2022-04-18 13:22:00
Message-ID: CA+TgmoY8zyrYBoBgZxccuPY9Wr1bzWve9=OwyMLSpfoc2F=8Ag@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, Apr 16, 2022 at 2:42 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> and this blew up with truly impressive thoroughness. The autovac
> launcher, logical replication launcher, and incoming backends all
> fail this assertion instantly, making it impossible to find out
> what else might be broken --- but I'm sure there is a lot.

OK, thanks for trying that.

> In the meantime, since it's clear that HaveRegisteredOrActiveSnapshot
> is failing to meet its contract, I'm going to go fix it. I think
> (based on the above argument) that what it intends to enforce is not
> really the system design we need, but it certainly isn't helping
> anyone that it enforces that design incorrectly.

+1.

--
Robert Haas
EDB: http://www.enterprisedb.com

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Japin Li 2022-04-18 13:36:01 Replace open mode with PG_BINARY_R/W/A macros
Previous Message Tomas Vondra 2022-04-18 12:36:00 Re: Column Filtering in Logical Replication