From: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> |
---|---|
To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
Cc: | Melih Mutlu <m(dot)melihmutlu(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Subscription statistics are not dropped at DROP SUBSCRIPTION in some cases |
Date: | 2023-06-19 01:19:24 |
Message-ID: | CAD21AoDCkcR2Y=caKQeWB1OMB=N0=G7JjXLBGktoMqzmf3mPwQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, Jun 17, 2023 at 6:45 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
> On Tue, May 16, 2023 at 8:00 PM Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
> >
> > On Thu, May 11, 2023 at 5:12 PM Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
> > >
> >
> > After thinking more about it, I realized that this is not a problem
> > specific to HEAD. ISTM the problem is that by commit 7b64e4b3, we drop
> > the stats entry of subscription that is not associated with a
> > replication slot for apply worker, but we missed the case where the
> > subscription is not associated with both replication slots for apply
> > and tablesync. So IIUC we should backpatch it down to 15.
> >
>
> I agree that it should be backpatched to 15.
>
> > Since in pg15, since we don't create the subscription stats at CREATE
> > SUBSCRIPTION time but do when the first error is reported,
> >
>
> AFAICS, the call to pgstat_create_subscription() is present in
> CreateSubscription() in 15 as well, so, I don't get your point.
IIUC in 15, pgstat_create_subscription() doesn't create the stats
entry. See commit e0b01429590.
Regards,
--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com
From | Date | Subject | |
---|---|---|---|
Next Message | torikoshia | 2023-06-19 02:24:29 | Re: Allow pg_archivecleanup to remove backup history files |
Previous Message | Tom Lane | 2023-06-19 01:10:59 | Re: [PATCH] hstore: Fix parsing on Mac OS X: isspace() is locale specific |