Re: pg_replslotdata - a tool for displaying replication slot information

From: Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>
To: Japin Li <japinli(at)hotmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: pg_replslotdata - a tool for displaying replication slot information
Date: 2021-11-25 04:52:15
Message-ID: CALj2ACV5pcVtBZdz7ydj7KSg=-jyxFVM-Ej2NttECti+VKTmBw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Nov 24, 2021 at 9:40 PM Japin Li <japinli(at)hotmail(dot)com> wrote:
>
>
> On Wed, 24 Nov 2021 at 23:59, Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> wrote:
> > On Wed, Nov 24, 2021 at 9:09 PM Bharath Rupireddy
> >> > Thoughts?
> >>
> >> Attaching the rebased v2 patch.
> >
> > On windows the previous patches were failing, fixed that in the v3
> > patch. I'm really sorry for the noise.
> >
>
> Cool! When I try to use it, there is an error for -v, --verbose option.
>
> px(at)ubuntu:~/Codes/postgres/Debug$ pg_replslotdata -v
> pg_replslotdata: invalid option -- 'v'
> Try "pg_replslotdata --help" for more information.
>
> This is because the getopt_long() missing 'v' in the third parameter.
>
> while ((c = getopt_long(argc, argv, "D:v", long_options, NULL)) != -1)

Thanks for taking a look at the patch, attaching v4.

There are many things that I could do in the patch, for instance, more
comments, documentation, code improvements etc. I would like to first
know what hackers think about this tool, and then start spending more
time on it.

Regards,
Bharath Rupireddy.

Attachment Content-Type Size
v4-0001-pg_replslotdata.patch application/octet-stream 21.9 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2021-11-25 04:57:09 Re: Skipping logical replication transactions on subscriber side
Previous Message Amul Sul 2021-11-25 04:51:40 Re: Deduplicate code updating ControleFile's DBState.