On Thu, Sep 15, 2022 at 10:39 AM Bharath Rupireddy <bharath.rupireddyforpostg...@gmail.com> wrote: > > On Mon, Jul 25, 2022 at 6:31 PM Bharath Rupireddy > <bharath.rupireddyforpostg...@gmail.com> wrote: > > > > Here's the v6 patch, a much simpler one - no changes to any of the > > existing function APIs. Please see the sample logs at [1]. There's a > > bit of duplicate code in the v6 patch, if the overall approach looks > > okay, I can remove that too in the next version of the patch. > > I modified the log_replication_commands description in guc_tables.c. > Please review the v7 patch further. >
I see that you have modified the patch to address the comments from Alvaro. Personally, I feel it would be better to add such a message at a centralized location instead of spreading these in different callers of slot acquire/release functionality to avoid getting these missed in the new callers in the future. However, if Alvaro and others think that the current style is better then we should go ahead and do it that way. I hope that we should be able to decide on this and get it into PG16. Anyone else would like to weigh in here? -- With Regards, Amit Kapila.
