Re: BUG #18876: HINT messages for mxid wrap-around say "drop stale slots", but that may not be appropriate

From: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: TAKATSUKA Haruka <harukat(at)sraoss(dot)co(dot)jp>, pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: BUG #18876: HINT messages for mxid wrap-around say "drop stale slots", but that may not be appropriate
Date: 2026-07-06 23:22:50
Message-ID: CAHGQGwEpG-9rsLZyqo49rcNDXAGjpL3LaiGc_iLYd4dTax+JAQ@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Tue, Jul 7, 2026 at 2:23 AM Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>
> On Thu, Jul 2, 2026 at 10:26 PM Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:
> > I've pushed the patch. Thanks!
>
> Hi,
>
> I'm a bit concerned about this patch, specifically this part:
>
> + <simpara>Unlike transaction ID wraparound, replication slots do not
> + directly hold back multixact cleanup. Dropping stale replication
> + slots is therefore not usually relevant to resolving multixact ID
> + wraparound problems.</simpara>
>
> The problem with the word "usually" here is that it leaves room for
> replication slots to be relevant in some corner case that isn't
> described, while asserting without explanation that they most often
> won't matter. I feel like that leaves users and DBAs in a very awkward
> position. The way it seems to me, either holding back the XID
> threshold can indirectly hold back the MXID threshold, or it can't. If
> it can, then the documentation before this patch was correct, and this
> shouldn't have been committed, and if it can't, then we don't need the
> word "usually" here.
>
> Now maybe that's a bit too simplistic. I'd be OK with a statement
> involving "usually" if it went on to explain convincingly why it
> doesn't normally happen and why it sometimes can under extraordinary
> circumstances. But without that, I feel this is making things less
> clear rather than more clear.

As far as I understand, there is no case where replication slots hold back
MXID cleanup. But I added "usually" only as a cautious wording choice.

If that my understanding is correct, I would like to update the docs to
remove "usually". Also, since "directly" in the same sentence might
cause similar confusion, it may be better to remove that as well.
So I am thinking of changing the text to something like this:

Unlike transaction ID wraparound, replication slots do not hold back
multixact cleanup. Dropping stale replication slots is therefore not
relevant to resolving multixact ID wraparound problems.

Thoughts?

--
Fujii Masao

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Matthias van de Meent 2026-07-06 23:51:32 Re: BUG #19519: REPACK can fail due to missing chunk for toast value
Previous Message Michael Paquier 2026-07-06 23:18:55 Re: BUG #19519: REPACK can fail due to missing chunk for toast value