Thread: make pg_createsubscriber option names more consistent

make pg_createsubscriber option names more consistent

From
Peter Eisentraut
Date:
I propose to rename the pg_createsubscriber option --socket-directory to 
--socketdir.  This would make it match the equivalent option in 
pg_upgrade.  (It even has the same short option '-s'.) 
pg_createsubscriber and pg_upgrade have a lot of common terminology and 
a similar operating mode, so it would make sense to keep this consistent.
Attachment

RE: make pg_createsubscriber option names more consistent

From
"Hayato Kuroda (Fujitsu)"
Date:
Dear Peter,

> I propose to rename the pg_createsubscriber option --socket-directory to
> --socketdir.  This would make it match the equivalent option in
> pg_upgrade.  (It even has the same short option '-s'.)
> pg_createsubscriber and pg_upgrade have a lot of common terminology and
> a similar operating mode, so it would make sense to keep this consistent.

+1. If so, should we say "default current dir." instead of "default current directory" in usage()
because pg_upgrade says like that?

Best regards,
Hayato Kuroda
FUJITSU LIMITED
 

Re: make pg_createsubscriber option names more consistent

From
"Euler Taveira"
Date:
On Wed, Jul 31, 2024, at 4:02 AM, Peter Eisentraut wrote:
I propose to rename the pg_createsubscriber option --socket-directory to 
--socketdir.  This would make it match the equivalent option in 
pg_upgrade.  (It even has the same short option '-s'.) 
pg_createsubscriber and pg_upgrade have a lot of common terminology and 
a similar operating mode, so it would make sense to keep this consistent.

WFM. 


--
Euler Taveira

Re: make pg_createsubscriber option names more consistent

From
Peter Eisentraut
Date:
On 31.07.24 11:15, Hayato Kuroda (Fujitsu) wrote:
> Dear Peter,
> 
>> I propose to rename the pg_createsubscriber option --socket-directory to
>> --socketdir.  This would make it match the equivalent option in
>> pg_upgrade.  (It even has the same short option '-s'.)
>> pg_createsubscriber and pg_upgrade have a lot of common terminology and
>> a similar operating mode, so it would make sense to keep this consistent.
> 
> +1. If so, should we say "default current dir." instead of "default current directory" in usage()
> because pg_upgrade says like that?

Committed with that change.  Thanks.