NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commit7558cc9

committed

Error out on send failure in walsender loop.

I changed the loop in 9.3 to use "goto send_failure" instead of "break" onerrors, but I missed this one case. It was a relatively harmless bug: ifthe flush fails once it will most likely fail again as soon as we try toflush the output again. But it's a bug nevertheless.Report and fix by Andres Freund.

1 parentb89e151 commit7558cc9Copy full SHA for 7558cc9

File tree

1 file changed

-1

lines changed

src/backend/replication
- walsender.c

1 file changed

-1

lines changed

`‎src/backend/replication/walsender.c`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1291,7 +1291,7 @@ WalSndLoop(void)`
`1291`	`1291`	`ping_sent= true;`
`1292`	`1292`	`/* Try to flush pending output to the client */`
`1293`	`1293`	`if (pq_flush_if_writable()!=0)`
`1294`		`-break;`
	`1294`	`+gotosend_failure;`
`1295`	`1295`	`}`
`1296`	`1296`	`}`
`1297`	`1297`

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!

Commit7558cc9

File tree

1 file changed

1 file changed

`‎src/backend/replication/walsender.c`

0 commit comments