- Notifications
You must be signed in to change notification settings - Fork5
Commitae90128
committed
Fix NOTIFY to cope with I/O problems, such as out-of-disk-space.
The LISTEN/NOTIFY subsystem got confused if SimpleLruZeroPage failed,which would typically happen as a result of a write() failure whileattempting to dump a dirty pg_notify page out of memory. Subsequently,all attempts to send more NOTIFY messages would fail with messages like"Could not read from file "pg_notify/nnnn" at offset nnnnn: Success".Only restarting the server would clear this condition. Per reports fromKevin Grittner and Christoph Berg.Back-patch to 9.0, where the problem was introduced during theLISTEN/NOTIFY rewrite.1 parent9e26326 commitae90128
1 file changed
+22
-5
lines changedLines changed: 22 additions & 5 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1285 | 1285 |
| |
1286 | 1286 |
| |
1287 | 1287 |
| |
| 1288 | + | |
1288 | 1289 |
| |
1289 | 1290 |
| |
1290 | 1291 |
| |
1291 | 1292 |
| |
1292 | 1293 |
| |
1293 | 1294 |
| |
1294 | 1295 |
| |
| 1296 | + | |
| 1297 | + | |
| 1298 | + | |
| 1299 | + | |
| 1300 | + | |
| 1301 | + | |
| 1302 | + | |
| 1303 | + | |
| 1304 | + | |
| 1305 | + | |
| 1306 | + | |
| 1307 | + | |
| 1308 | + | |
1295 | 1309 |
| |
1296 |
| - | |
| 1310 | + | |
1297 | 1311 |
| |
1298 | 1312 |
| |
1299 | 1313 |
| |
| |||
1305 | 1319 |
| |
1306 | 1320 |
| |
1307 | 1321 |
| |
1308 |
| - | |
| 1322 | + | |
1309 | 1323 |
| |
1310 | 1324 |
| |
1311 | 1325 |
| |
| |||
1331 | 1345 |
| |
1332 | 1346 |
| |
1333 | 1347 |
| |
1334 |
| - | |
1335 |
| - | |
| 1348 | + | |
| 1349 | + | |
1336 | 1350 |
| |
1337 | 1351 |
| |
1338 | 1352 |
| |
| |||
1342 | 1356 |
| |
1343 | 1357 |
| |
1344 | 1358 |
| |
1345 |
| - | |
| 1359 | + | |
1346 | 1360 |
| |
1347 | 1361 |
| |
1348 | 1362 |
| |
1349 | 1363 |
| |
1350 | 1364 |
| |
| 1365 | + | |
| 1366 | + | |
| 1367 | + | |
1351 | 1368 |
| |
1352 | 1369 |
| |
1353 | 1370 |
| |
|
0 commit comments
Comments
(0)