Re: Adjust the description of OutputPluginCallbacks in pg-doc
Amit Kapila <amit.kapila16@gmail.com>
From: Amit Kapila <amit.kapila16@gmail.com>
To: "wangw.fnst@fujitsu.com" <wangw.fnst@fujitsu.com>
Cc: "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>
Date: 2023-01-20T04:40:29Z
Lists: pgsql-hackers
On Fri, Jan 20, 2023 at 8:03 AM wangw.fnst@fujitsu.com <wangw.fnst@fujitsu.com> wrote: > > On Thurs, Jan 19, 2023 at 19:18 PM Amit Kapila <amit.kapila16@gmail.com> wrote: > > On Wed, Jan 11, 2023 at 4:20 PM wangw.fnst@fujitsu.com > > <wangw.fnst@fujitsu.com> wrote: > > > > > > When I was reading the "Logical Decoding Output Plugins" chapter in pg-doc > > [1], > > > I think in the summary section, only the callback message_cb is not described > > > whether it is required or optional, and the description of callback > > > stream_prepare_cb seems inaccurate. > > > > > > And after the summary section, I think only the callback stream_xxx_cb > > section > > > and the callback truncate_cb section are not described this tag (are they > > > required or optional). > > > > > > I think we could improve these to be more reader friendly. So I tried to write > > > a patch for these and attach it. > > > > > > Any thoughts? > > > > > > > This looks mostly good to me. I have made minor adjustments in the > > attached. Do those make sense to you? > > Thanks for your improvement. > It makes sense to me. > Pushed. -- With Regards, Amit Kapila.
Commits
-
Improve the description of Output Plugin Callbacks.
- 7670add9f43a 16.0 landed