On Sat, Jan 7, 2017 at 12:04 AM, Magnus Hagander <magnus@hagander.net> wrote: > On Wed, Jan 4, 2017 at 10:43 AM, Magnus Hagander <magnus@hagander.net> > wrote: >> Meh, just as I was going to respond "committed" I noticed this second >> round of review comments. Apologies, pushed without that. >> >> I agree on the change with includewal/streamwal. That was already the case >> in the existing code of course, but that doesn't mean it couldn't be made >> better. I'll take a look at doing that as a separate patch. >> > > Here's a patch that does this. Does this match what you were thinking?
Yes, that's it. I have looked at the patch in details and that looks correct to me.
OK. Pushed. I agree it made it more readable, if nothing else.