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.
