NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit26f9e21

committed

Temporarily skip recovery deadlock test in back branches.

The recovery deadlock test has a timing issue that was fixed in5136967 inHEAD. Unfortunately the same fix doesn't quite work in the back branches: 1)adjust_conf() doesn't exist, which is easy enough to work around 2) a restartcleares the recovery conflict stats < 15.These issues can be worked around, but given the upcoming set of minorreleases, skip the problematic test for now. The buildfarm doesn't showfailures in other parts of 031_recovery_conflict.pl.Discussion:https://postgr.es/m/20220506155827.dfnaheq6ufylwrqf@alap3.anarazel.deBackpatch: 10-14

1 parentfbf659b commit26f9e21Copy full SHA for 26f9e21

File tree

1 file changed

-0

lines changed

src/test/recovery/t
- 031_recovery_conflict.pl

1 file changed

-0

lines changed

`‎src/test/recovery/t/031_recovery_conflict.pl`

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -228,6 +228,10 @@`
`228`	`228`
`229`	`229`
`230`	`230`	`## RECOVERY CONFLICT 5: Deadlock`
	`231`	`+SKIP:`
	`232`	`+{`
	`233`	`+skip"disabled until after minor releases, due to instability";`
	`234`	`+`
`231`	`235`	`$sect ="startup deadlock";`
`232`	`236`	`$expected_conflicts++;`
`233`	`237`
`@@ -286,6 +290,7 @@`
`286`	`290`
`287`	`291`	`# clean up for next tests`
`288`	`292`	`$node_primary->safe_psql($test_db,qq[ROLLBACK PREPARED 'lock';]);`
	`293`	`+}`
`289`	`294`
`290`	`295`
`291`	`296`	`# Check that expected number of conflicts show in pg_stat_database. Needs to`

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!

Commit26f9e21

File tree

1 file changed

1 file changed

`‎src/test/recovery/t/031_recovery_conflict.pl`

0 commit comments