On Tue, Nov 8, 2022 at 3:20 PM Thomas Munro <thomas.mu...@gmail.com> wrote: > On Sun, Nov 6, 2022 at 12:01 PM Nathan Bossart <nathandboss...@gmail.com> > 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.
