forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit9b95a36
committed
Make crash recovery ignore recovery_min_apply_delay setting.
In v11 or before, this setting could not take effect in crash recoverybecause it's specified in recovery.conf and crash recovery alwaysstarts without recovery.conf. But commit2dedf4d integratedrecovery.conf into postgresql.conf and which unexpectedly allowedthis setting to take effect even in crash recovery. This is definitelynot good behavior.To fix the issue, this commit makes crash recovery always ignorerecovery_min_apply_delay setting.Back-patch to v12 where the issue was added.Author: Fujii MasaoReviewed-by: Michael PaquierDiscussion:https://postgr.es/m/CAHGQGwEyD6HdZLfdWc+95g=VQFPR4zQL4n+yHxQgGEGjaSVheQ@mail.gmail.comDiscussion:https://postgr.es/m/e445616d-023e-a268-8aa1-67b8b335340c@pgmasters.net1 parent89403ed commit9b95a36
2 files changed
+6
-1
lines changedLines changed: 2 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
4209 | 4209 |
| |
4210 | 4210 |
| |
4211 | 4211 |
| |
4212 |
| - | |
| 4212 | + | |
| 4213 | + | |
4213 | 4214 |
| |
4214 | 4215 |
| |
4215 | 4216 |
| |
|
Lines changed: 4 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
5977 | 5977 |
| |
5978 | 5978 |
| |
5979 | 5979 |
| |
| 5980 | + | |
| 5981 | + | |
| 5982 | + | |
| 5983 | + | |
5980 | 5984 |
| |
5981 | 5985 |
| |
5982 | 5986 |
| |
|
0 commit comments
Comments
(0)