Re: pgsql: Further further fix pg_upgrade crossversion test for adminpack.

From: Jeff Davis <pgsql(at)j-davis(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Further further fix pg_upgrade crossversion test for adminpack.
Date: 2024-03-09 07:48:08
Message-ID: bca97aecb50b2026b7dbc26604bf31861c819a64.camel@j-davis.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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 Content-Type Size
0001-Fix-cross-version-pg_upgrade-test.patch text/x-patch 1.1 KB

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2024-03-09 17:49:57 Re: pgsql: Further further fix pg_upgrade crossversion test for adminpack.
Previous Message Michael Paquier 2024-03-09 06:48:26 pgsql: Document units of "timeout" in ConditionVariableTimedSleep()