Re: pg_recvlogical cannot create slots with failover=true - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: pg_recvlogical cannot create slots with failover=true
Date
Msg-id a28f66df-1354-4709-8d63-932ded4cac35@eisentraut.org
Whole thread Raw
In response to Re: pg_recvlogical cannot create slots with failover=true  (Masahiko Sawada <sawada.mshk@gmail.com>)
Responses Re: pg_recvlogical cannot create slots with failover=true
List pgsql-hackers
On 07.04.25 21:15, Masahiko Sawada wrote:
> On Sun, Apr 6, 2025 at 7:19 PM Zhijie Hou (Fujitsu)
> <houzj.fnst@fujitsu.com> wrote:
>>
>> On Sat, Apr 5, 2025 at 1:45 AM Masahiko Sawada wrote:
>>
>> Hi,
>>
>>> Thank you for updating the patch! Pushed with small cosmetic changes.
>>
>> Thanks for pushing the feature !
>>
>> I noticed one typo in the doc and here is a tiny patch to fix it.
>>
>> -        The <option>--two-phase</option> and <option>--falover</option> options
>> +        The <option>--two-phase</option> and <option>--failover</option> options
> 
> Thank you for the report and the patch! Pushed.

I wonder if the option name --failover is ideal.  To me, it sounds like 
an action "do a failover!".  Also consider that pg_recvlogical has other 
action options such as --start and --create-slot, so it sounds a bit 
like those.

Maybe something like --enable-failover would be better?




pgsql-hackers by date:

Previous
From: Steven Niu
Date:
Subject: Re: Support tid range scan in parallel?
Next
From: Peter Smith
Date:
Subject: Re: Reduce TupleHashEntryData struct size by half