On Tue, Nov 8, 2022 at 3:20 PM Thomas Munro <[email protected]> wrote:
> On Sun, Nov 6, 2022 at 12:01 PM Nathan Bossart <[email protected]> 
> wrote:
> > Here is a new version of the patch that addresses this feedback.
>
> This looks pretty good to me.  Thanks for picking it up!  I can live
> with the change to use a global variable; it seems we couldn't quite
> decide what the scope was for moving stuff into a struct (ie various
> pre-existing stuff representing the walreceiver's state), but I'm OK
> with considering that as a pure refactoring project for a separate
> thread.  That array should be "static", though.

And with that change and a pgindent, pushed.


Reply via email to