On Fri, Apr 16, 2021 at 8:22 AM Justin Pryzby <pry...@telsasoft.com> wrote: > > On Thu, Apr 15, 2021 at 02:46:35PM +0530, Amit Kapila wrote: > > On Thu, Apr 15, 2021 at 1:13 PM Masahiko Sawada <sawada.m...@gmail.com> > > wrote: > > > > > > On Thu, Apr 15, 2021 at 3:22 PM Amit Kapila <amit.kapil...@gmail.com> > > > wrote: > > > > > > > Thank you for updating the patch. > > > > > > I have one question on the doc change: > > > > > > + so the counter is not incremented for subtransactions. Note that > > > this > > > + includes the transactions streamed and or spilled. > > > + </para></entry> > > > > > > The patch uses the sentence "streamed and or spilled" in two places. > > > You meant “streamed and spilled”? Even if it actually means “and or”, > > > using "and or” (i.g., connecting “and” to “or” by a space) is general? > > > I could not find we use it other places in the doc but found we're > > > using "and/or" instead. > > > > > > > I changed it to 'and/or' and made another minor change. > > I'm suggesting some doc changes. If these are fine, I'll include in my next > round of doc review, in case you don't want to make another commit just for > that. >
I am fine with your proposed changes. There are one or two more patches in this area. I can include your suggestions along with those if you don't mind? -- With Regards, Amit Kapila.