make pg_createsubscriber option names more consistent

Started by Peter Eisentrautover 1 year ago4 messageshackers
Jump to latest
#1Peter Eisentraut
peter_e@gmx.net

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.

Attachments:

0001-pg_createsubscriber-Rename-option-socket-directory-t.patchtext/plain; charset=UTF-8; name=0001-pg_createsubscriber-Rename-option-socket-directory-t.patchDownload+12-13
#2Hayato Kuroda (Fujitsu)
kuroda.hayato@fujitsu.com
In reply to: Peter Eisentraut (#1)
RE: make pg_createsubscriber option names more consistent

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

#3Euler Taveira
euler@eulerto.com
In reply to: Peter Eisentraut (#1)
Re: make pg_createsubscriber option names more consistent

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
EDB https://www.enterprisedb.com/

#4Peter Eisentraut
peter_e@gmx.net
In reply to: Hayato Kuroda (Fujitsu) (#2)
Re: make pg_createsubscriber option names more consistent

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.