| 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: | Whole Thread | Raw Message | 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 |
| 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. |