Re: Enhance 'pg_createsubscriber' to retrieve databases automatically when no database is provided.
Shubham Khanna <khannashubham1197@gmail.com>
From: Shubham Khanna <khannashubham1197@gmail.com>
To: Ajin Cherian <itsajin@gmail.com>
Cc: Ashutosh Bapat <ashutosh.bapat.oss@gmail.com>, "Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>,
Peter Smith <smithpb2250@gmail.com>, Amit Kapila <amit.kapila16@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-03-10T07:09:04Z
Lists: pgsql-hackers
On Mon, Mar 10, 2025 at 11:04 AM Ajin Cherian <itsajin@gmail.com> wrote: > > On Mon, Mar 10, 2025 at 3:58 PM Ashutosh Bapat > <ashutosh.bapat.oss@gmail.com> wrote: > > > > On Thu, Mar 6, 2025 at 9:18 AM Ajin Cherian <itsajin@gmail.com> wrote: > > > > > + Subscription names, publication names, and replication slot names are > > > + automatically generated. Cannot be used together with > > > + <option>--database</option>, <option>--publication</option>, > > > + <option>--replication-slot</option> or <option>--subscription</option>. > > > > > > Don't start the sentence with "Cannot". Change the sentence to "This > > > option cannot be used together with ..." > > > similar sentences used in 3 other places below this as well. Change all of them. > > > > I didn't find any instance of "Cannot be" in the *.sgml files, but I > > find some instances of "Can be". So it seems we allow such constructs > > in the documentation. Any reasons for suggesting this change? > > > > I just don't think it is correct English to start a sentence with > "Cannot be". I checked with grammarly as well. > Fixed. The attached patch at [1] contains the required changes. [1] - https://www.postgresql.org/message-id/CAHv8Rj%2B37ja589BzqB5bz0ZYGWb5gpnP9of8SoqKc%3DDqLmvxBg%40mail.gmail.com Thanks and regards, Shubham Khanna.
Commits
-
Cosmetic fixes for pg_createsubscriber's -all option.
- d438515c29c3 18.0 landed
-
pg_createsubscriber: Add '--all' option.
- fb2ea12f42b9 18.0 landed