On Thu, Jun 2, 2022 at 9:58 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Fri, May 27, 2022 at 1:04 PM houzj.f...@fujitsu.com > <houzj.f...@fujitsu.com> wrote: > > > > On Friday, May 27, 2022 1:54 PM Justin Pryzby <pry...@telsasoft.com> wrote: > > > > > > On Fri, May 27, 2022 at 11:17:00AM +0530, Amit Kapila wrote: > > > > On Tue, May 24, 2022 at 11:03 AM houzj.f...@fujitsu.com > > > <houzj.f...@fujitsu.com> wrote: > > > > > > > > > > On Friday, May 20, 2022 11:06 AM Amit Kapila <amit.kapil...@gmail.com> > > > wrote: > > > > > > > > > > Thanks for pointing it out. Here is the new version patch which add > > > > > this > > > version check. > > > > > > > > I have added/edited a few comments and ran pgindent. The attached > > > > looks good to me. I'll push this early next week unless there are more > > > > comments/suggestions. > > > > > > A minor doc review. > > > Note that I also sent some doc comments at > > > 20220519120724.go19...@telsasoft.com. > > > > > > + lists among publications in which case <command>ALTER > > > PUBLICATION</command> > > > + command will be successful but later the WalSender in publisher > > > + or the > > > > > > COMMA in which > > > > > > remove "command" ? > > > > > > s/in publisher/on the publisher/ > > > > > > + Subscription having several publications in which the same table has > > > been > > > + published with different column lists is not supported. > > > > > > Either "Subscriptions having .. are not supported"; or, "A subscription > > > having .. > > > is not supported". > > > > Thanks for the comments. Here is the new version patch set which fixes > > these. > > > > I have pushed the bug-fix patch. I'll look at the language > improvements patch next.
I noticed the patch "0001-language-fixes-on-HEAD-from-Justin.patch" says: @@ -11673,7 +11673,7 @@ prosrc => 'pg_show_replication_origin_status' }, # publications -{ oid => '6119', descr => 'get information of tables in a publication', +{ oid => '6119', descr => 'get information about tables in a publication', ~~~ But, this grammar website [1] says: What Does Of Mean As defined by Cambridge dictionary Of is basically used “to show possession, belonging, or origin”. What Does About Mean Similarly about primarily indicates ‘On the subject of; concerning’ as defined by the Oxford dictionary. Or about in brief highlights some fact ‘on the subject of, or connected with’ The main difference between of and about is that of implies a possessive quality while about implies concerning or on the subject of something. ~~~ >From which I guess 1. 'get information of tables in a publication' ~= 'get information belonging to tables in a publication' 2. 'get information about tables in a publication' ~= 'get information on the subject of tables in a publication' The 'pg_publication_tables' view contains various attributes (tablename, attnames, rowfilter, etc) BELONGING TO each table of the publication, so the current description (using 'of') was already the more accurate one wasn't it? ------ [1] https://pediaa.com/difference-between-of-and-about/ Kind Regards, Peter Smith. Fujitsu Australia