Re: repeated decoding of prepared transactions

On Tue, Mar 2, 2021 at 12:43 PM vignesh C <vignesh21(at)gmail(dot)com> wrote:
>
> On Tue, Mar 2, 2021 at 9:33 AM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> >
> > On Tue, Mar 2, 2021 at 8:20 AM vignesh C <vignesh21(at)gmail(dot)com> wrote:
> > >
> > >
> > > I have a minor comment regarding the below:
> > > + <row>
> > > + <entry role="catalog_table_entry"><para role="column_definition">
> > > + <structfield>two_phase</structfield> <type>bool</type>
> > > + </para>
> > > + <para>
> > > + True if two-phase commits are enabled on this slot.
> > > + </para></entry>
> > > + </row>
> > >
> > > Can we change something like:
> > > True if the slot is enabled for decoding prepared transaction
> > > information. Refer link for more information.(link should point where
> > > more detailed information is available for two-phase in
> > > pg_create_logical_replication_slot).
> > >
> > > Also there is one small indentation in that line, I think there should
> > > be one space before "True if....".
> > >
> >
> > Okay, fixed these but I added a slightly different description. I have
> > also added the parameter description for
> > pg_create_logical_replication_slot in docs and changed the comments at
> > various places in the code. Apart from that ran pgindent. The patch
> > looks good to me now. Let me know what do you think?
>
> Patch applies cleanly, make check and make check-world passes. I did
> not find any other issue. The patch looks good to me.
>

Thanks, I have pushed this patch.

--
With Regards,
Amit Kapila.