Re: pg_recvlogical requires -d but not described on the documentation

From: "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>
To: Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>
Cc: "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com>, Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: pg_recvlogical requires -d but not described on the documentation
Date: 2025-03-18 06:10:52
Message-ID: CAKFQuwYoBUEYV218EYZeVn7S7S=kdoydw2PnAWszk0aTeS+k6A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Monday, March 17, 2025, Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> wrote:

>
> On 2025/03/12 14:59, David G. Johnston wrote:
>
>> On Monday, February 24, 2025, Hayato Kuroda (Fujitsu) <
>> kuroda(dot)hayato(at)fujitsu(dot)com <mailto:kuroda(dot)hayato(at)fujitsu(dot)com>> wrote:
>>
>>
>> OK, so I will exclude the point in the thread. The patch I've posted
>> contains all of fixes
>> which is required.
>>
>>
>> The patch fixes the synopsis and the mention of the default value. Only
>> the later is required. I cannot figure out a policy that would alter the
>> synopsis in the proposed manner. I’d suggest leaving it alone for now and
>> tweak any of the option descriptions that may need clarification.
>>
>
> I agree that the synopsis doesn't need to be updated. Attached patch
> clarifies
> the required options for each action in the documentation. Thought?

Will look again tomorrow but seems ok aside from needing an editing pass.

The usage section for help probably needs a look as well.

>
> BTW, I'm curious why --dbname isn't required for the --drop-slot action.
> When I ran
> pg_recvlogical --drop-slot without --dbname, I got the following error:
>
> pg_recvlogical: error: could not establish database-specific replication
> connection
>

That would be a bug.

I think this is too:

set_pglocale_pgservice(argv[0], PG_TEXTDOMAIN("pg_basebackup"));

Though I’m not sure what it is doing.

David J.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Ashutosh Bapat 2025-03-18 06:15:47 Re: Fix couple of typos
Previous Message Stepan Neretin 2025-03-18 06:03:38 Optimize truncation logic to reduce AccessExclusive lock impact