Re: pgsql: Further further fix pg_upgrade crossversion test for adminpack. - Mailing list pgsql-committers

From Jeff Davis
Subject Re: pgsql: Further further fix pg_upgrade crossversion test for adminpack.
Date
Msg-id bca97aecb50b2026b7dbc26604bf31861c819a64.camel@j-davis.com
Whole thread Raw
In response to pgsql: Further further fix pg_upgrade crossversion test for adminpack.  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: pgsql: Further further fix pg_upgrade crossversion test for adminpack.
List pgsql-committers
On Mon, 2024-03-04 at 18:10 +0000, Tom Lane wrote:
> Further further fix pg_upgrade crossversion test for adminpack.

Ever since this commit the cross-version upgrade test is failing (for
me, at least) with:

# Running: psql -X -v ON_ERROR_STOP=1 -c drop database if exists
contrib_regression_adminpack;
drop database if exists regression_adminpack -d port=53977
host=/tmp/EK6UT_TufI dbname='postgres'
ERROR:  DROP DATABASE cannot run inside a transaction block

It looks like when you added another command, the two were joined with
";\n", which ends up running the commands in a transaction block, which
doesn't work for DROP DATABASE.

I'm not sure how this test is succeeding for others, so perhaps I'm
doing something wrong?

Patch attached, though I'm not particularly great with perl and the
array flattening in my implementation might be too magical.

Regards,
    Jeff Davis


Attachment

pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: Document units of "timeout" in ConditionVariableTimedSleep()
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Further further fix pg_upgrade crossversion test for adminpack.