On Fri, Jul 25, 2025 at 10:53 PM cca5507 <cca5...@qq.com> wrote:
>
> > For example, we could add a check like the following to
> > 013_crash_restart.pl.
> > Thoughts?
> >
> > -----------------
> > is($node->safe_psql('postgres',
> > "SELECT count(*) = 1 FROM pg_stat_activity WHERE backend_type =
> > 'logical replication launcher'"),
> > 't',
> > 'logical replication launcher is running after crash');
> > -----------------Patch attached. I confirmed that the test fails when applied to a version before commit b5d084c5353 (i.e., before the bug was fixed), and it passes on HEAD with the patch applied. > Agree, but we need note that the logical replication launcher won't be > registered in some case. (see ApplyLauncherRegister()) Since 013_crash_restart.pl doesn't set any parameters that would prevent the logical replication launcher from starting, I think we don't need to worry about that. No?? Regards, -- Fujii Masao
v1-0001-Add-regression-test-for-background-worker-restart.patch
Description: Binary data
