Amit, thanks for looking into this!

On Sun, Nov 20, 2022 at 11:38 PM Amit Kapila <amit.kapil...@gmail.com>
wrote:

> On Mon, Nov 21, 2022 at 12:45 PM sirisha chamarthi
> <sirichamarth...@gmail.com> wrote:
> >
> > Hi Hackers,
> >
> > The comments atop seem to indicate that it is only showing active
> replication slots. The comment is ambiguous as it also shows all the slots
> including lost and inactive slots. Attached a small patch to fix it.
> >
>
> I agree that it is a bit confusing. How about "SQL SRF showing all
> replication slots that currently exist on the database cluster"?
>

Looks good to me. Attached a patch for the same.


>
> --
> With Regards,
> Amit Kapila.
>

Attachment: 0002-Fix-atop-pg_get_replication_slots-function-to-reflec.patch
Description: Binary data

Reply via email to