NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitae55d9f

committed

Remove prepared transactions from main isolation test schedule.

There is no point in running this test when prepared transactions are disabled,which is the default. New make targets that include the test are provided. Thiswill save some useless waste of cycles on buildfarm machines.Backpatch to 9.1 where these tests were introduced.

1 parent8ca03aa commitae55d9fCopy full SHA for ae55d9f

File tree

2 files changed

+10

-1

lines changed

src/test/isolation
- Makefile
- isolation_schedule

2 files changed

+10

-1

lines changed

`‎src/test/isolation/Makefile`

Lines changed: 10 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -72,3 +72,13 @@ installcheck: all`
`72`	`72`
`73`	`73`	`check: all`
`74`	`74`	`./pg_isolation_regress --temp-install=./tmp_check --inputdir=$(srcdir) --top-builddir=$(top_builddir) --schedule=$(srcdir)/isolation_schedule`
	`75`	`+`
	`76`	`+# Versions of the check tests that include the prepared_transactions test`
	`77`	`+# It only makes sense to run these if set up to use prepared transactions,`
	`78`	`+# via TEMP_CONFIG for the check case, or via the postgresql.conf for the`
	`79`	`+# installcheck case.`
	`80`	`+installcheck-prepared-txns: all`
	`81`	`+./pg_isolation_regress --psqldir='$(PSQLDIR)' --inputdir=$(srcdir) --schedule=$(srcdir)/isolation_schedule prepared-transactions`
	`82`	`+`
	`83`	`+check-prepared-txns: all`
	`84`	`+./pg_isolation_regress --temp-install=./tmp_check --inputdir=$(srcdir) --top-builddir=$(top_builddir) --schedule=$(srcdir)/isolation_schedule prepared-transactions`

`‎src/test/isolation/isolation_schedule`

Lines changed: 0 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,6 @@ test: ri-trigger`
`9`	`9`	`test: partial-index`
`10`	`10`	`test: two-ids`
`11`	`11`	`test: multiple-row-versions`
`12`		`-test: prepared-transactions`
`13`	`12`	`test: fk-contention`
`14`	`13`	`test: fk-deadlock`
`15`	`14`	`test: fk-deadlock2`

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!

Commitae55d9f

File tree

2 files changed

2 files changed

`‎src/test/isolation/Makefile`

`‎src/test/isolation/isolation_schedule`

0 commit comments