On Fri, Dec 23, 2022 at 12:57 PM David Christensen
<david.christensen@crunchydata.com> wrote:
>
> On Wed, Dec 21, 2022 at 5:47 AM Bharath Rupireddy
> <bharath.rupireddyforpostgres@gmail.com> wrote:
[snip]
> > 2. +$node->init(extra => ['-k'], allows_streaming => 1);
> > When enabled with allows_streaming, there are a bunch of things that
> > happen to the node while initializing, I don't think we need all of
> > them for this.
>
> I think the "allows_streaming" was required to ensure the WAL files
> were preserved properly, and was the approach we ended up taking
> rather than trying to fail the archive_command or other approaches I'd
> taken earlier. I'd rather keep this if we can, unless you can propose
> a different approach that would continue to work in the same way.
Confirmed that we needed this in order to create the replication slot,
so this /is/ required for the test to work.
Enclosing v11 with yours and Michael's latest feedback.
Best,
David