Re: Fix "unexpected logical decoding status change" error; from concurrent logical decoding activation

From: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
To: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
Cc: Srinath Reddy Sadipiralla <srinath2133(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Fix "unexpected logical decoding status change" error; from concurrent logical decoding activation
Date: 2026-07-30 21:48:39
Message-ID: CAD21AoDFpgsqN0b0tyg6iLJ=F7ntYua_2y6x4UUfk9WQu3WBsA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Jul 30, 2026 at 1:58 PM Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
>
> On Thu, Jul 30, 2026 at 2:00 AM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> >
> > On Thu, Jul 30, 2026 at 12:37 PM Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
> > >
> > > Thank you for testing it. It seems to be reproducible also on CI, and
> > > I believe I've fixed the issue.
> > >
> > > I've attached the patch. Please review it.
> > >
> >
> > LGTM.
>
> Thank you for reviewing the patch. Pushed.
>
> ... but some buildfarm members are unhappy. I'm working on it.
>

I've reproduced the issue on local and confirmed the root cause; two
background psql sessions in the TAP test run with ON_ERROR_STOP=1 and
got canceled by pg_cancel_backend(), so psql exited as soon as the
cancelation error arrived. I've attached the patch that fixes the
problem. I'm waiting for CI to check if the tests still pass on other
environments.

Regards,

--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com

Attachment Content-Type Size
0001-Fix-background-psql-session-cleanup-in-051_effective.patch text/x-patch 2.8 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Zsolt Parragi 2026-07-30 22:02:31 Re: datachecksums: handle invalid and dropped databases during enable
Previous Message Greg Lamberson 2026-07-30 21:46:11 Re: Extensible sync handler registration (register_sync_handler)