On Wed, Oct 15, 2025 at 07:30:39PM +0530, Kevin K Biju wrote: > Thanks for the context. I missed the earlier discussions about the same > issue. Using unlink in the error paths makes sense to me. There is an edge > case in my mind, in case unlink fails as well, and we end up in the same > condition; however, the chance of that occurring is sufficiently low.
My suspicion is that an unlink() failure would link to much more problems, including what could be crashes in critical sections. That would mean a crash, where the state.tmp file would get cleaned up by recovery. What we have now in the tree should be (I hope!) a good balance. That was 6 years ago, and it is very easy to miss, so no worries. Even in my case, I've recalled the previous discussion after one night of sleep, only because I've participated in it and because the automated truncation you have suggested was itching me. Somewhat. > Thanks for the quick response! No problem. -- Michael
signature.asc
Description: PGP signature
