Re: tweak to a few index tests to hits ambuildempty() routine.

On Sun, May 22, 2022 at 04:24:16PM +1200, Thomas Munro wrote:
> On Sat, May 21, 2022 at 6:15 PM Noah Misch <noah@leadboat.com> wrote:
> > On Mon, Apr 25, 2022 at 10:05:08AM -0400, Tom Lane wrote:
> > > Alvaro Herrera <alvherre@alvh.no-ip.org> writes:
> > > > Hmm, so 027_stream_regress.pl is not prepared to deal with any unlogged
> > > > tables that may be left in the regression database (which is what my
> > > > spgist addition did).  I first tried doing a TRUNCATE of the unlogged
> > > > table, but that doesn't work either, and it turns out that the
> > > > regression database does not have any UNLOGGED relations.  Maybe that's
> > > > something we need to cater for, eventually, but for now dropping the
> > > > table suffices.  I have pushed that.
> > >
> > > It does seem like the onus should be on 027_stream_regress.pl to
> > > deal with that, rather than restricting what the core tests can
> > > leave behind.
> >
> > Yeah.  Using "pg_dumpall --no-unlogged-table-data", as attached, suffices.
> 
>                 'pg_dumpall', '-f', $outputdir . '/primary.dump',
> -               '--no-sync', '-p', $node_primary->port
> +               '--no-sync',  '-p', $node_primary->port,
> +               '--no-unlogged-table-data'    # if unlogged, standby
> has schema only
> 
> LGTM, except for the stray extra whitespace.

perltidy contributes the prior-line whitespace change.