NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitb936d92

committed

Report libpq errors correctly if session setup or teardown steps fail in

isolation regression tests.Alvaro committed these fixes to master branch on Tue Jul 29th, as part ofNoah Misch's patch. The rest of that patch is not needed on 9.1, but thispart should be backpatched.

1 parentb9585e8 commitb936d92Copy full SHA for b936d92

File tree

1 file changed

-2

lines changed

src/test/isolation
- isolationtester.c

1 file changed

-2

lines changed

`‎src/test/isolation/isolationtester.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -286,7 +286,7 @@ run_permutation(TestSpec * testspec, int nsteps, Step ** steps)`
`286`	`286`	`{`
`287`	`287`	`fprintf(stderr,"setup of session %s failed: %s",`
`288`	`288`	`testspec->sessions[i]->name,`
`289`		`-PQerrorMessage(conns[0]));`
	`289`	`+PQerrorMessage(conns[i]));`
`290`	`290`	`exit_nicely();`
`291`	`291`	`}`
`292`	`292`	`PQclear(res);`
`@@ -333,7 +333,7 @@ run_permutation(TestSpec * testspec, int nsteps, Step ** steps)`
`333`	`333`	`{`
`334`	`334`	`fprintf(stderr,"teardown of session %s failed: %s",`
`335`	`335`	`testspec->sessions[i]->name,`
`336`		`-PQerrorMessage(conns[0]));`
	`336`	`+PQerrorMessage(conns[i]));`
`337`	`337`	`/* don't exit on teardown failure */`
`338`	`338`	`}`
`339`	`339`	`PQclear(res);`

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!

Commitb936d92

File tree

1 file changed

1 file changed

`‎src/test/isolation/isolationtester.c`

0 commit comments