forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitb94ce6e
committed
Move postmaster's RemovePgTempFiles call to a less randomly chosen place.
There is no reason to do this as early as possible in postmaster startup,and good reason not to do it until we have completely created thepostmaster's lock file, namely that it might contribute to pg_ctl thinkingthat postmaster startup has timed out. (This would require a ratherunusual amount of time to be spent scanning temp file directories, but wehave at least one field report of it happening reproducibly.)Back-patch to 9.1. Before that, pg_ctl didn't wait for additional info tobe added to the lock file, so it wasn't a problem.Note that this is not a complete fix to the slow-start issue in 9.1,because we still had identify_system_timezone being run during postmasterstart in 9.1. But that's at least a reasonably well-defined delay, withan easy workaround if needed, whereas the temp-files scan is not sopredictable and cannot be avoided.1 parentefae465 commitb94ce6e
1 file changed
+6
-6
lines changedLines changed: 6 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
826 | 826 |
| |
827 | 827 |
| |
828 | 828 |
| |
829 |
| - | |
830 |
| - | |
831 |
| - | |
832 |
| - | |
833 |
| - | |
834 |
| - | |
835 | 829 |
| |
836 | 830 |
| |
837 | 831 |
| |
| |||
1099 | 1093 |
| |
1100 | 1094 |
| |
1101 | 1095 |
| |
| 1096 | + | |
| 1097 | + | |
| 1098 | + | |
| 1099 | + | |
| 1100 | + | |
| 1101 | + | |
1102 | 1102 |
| |
1103 | 1103 |
| |
1104 | 1104 |
| |
|
0 commit comments
Comments
(0)