NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitc8a61e3

committed

Further fix pg_upgrade crossversion test for adminpack.

The DROP DATABASE step needs an "if exists" option, as the oldestbranches we test don't have the contrib_regression_adminpack DB.Also remove unnecessary command to drop the extension from theregression database; no version has installed it there duringbuildfarm testing.Discussion:https://postgr.es/m/0CFB76D0-0510-48B2-9916-1199F93BC28C@yesql.se

1 parent084cff7 commitc8a61e3Copy full SHA for c8a61e3

File tree

1 file changed

-3

lines changed

src/test/perl/PostgreSQL/Test
- AdjustUpgrade.pm

1 file changed

-3

lines changed

`‎src/test/perl/PostgreSQL/Test/AdjustUpgrade.pm`

Lines changed: 1 addition & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -109,10 +109,8 @@ sub adjust_database_contents`
`109`	`109`	`# we removed the adminpack extension in v17`
`110`	`110`	`if ($old_version < 17)`
`111`	`111`	`{`
`112`		`-_add_st($result,'regression',`
`113`		`-'drop extension if exists adminpack');`
`114`	`112`	`_add_st($result,'postgres',`
`115`		`-'drop database contrib_regression_adminpack');`
	`113`	`+'drop databaseif existscontrib_regression_adminpack');`
`116`	`114`	`delete($dbnames{'contrib_regression_adminpack'});`
`117`	`115`	`}`
`118`	`116`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitc8a61e3

File tree

1 file changed

1 file changed

`‎src/test/perl/PostgreSQL/Test/AdjustUpgrade.pm`

0 commit comments