On Wed, Mar 18, 2020 at 4:25 PM Andres Freund <and...@anarazel.de> wrote: > I don't see a valid reason for that though - if anything it's dangerous, > because we're not persistently saving the slot. It should fail the > checkpoint imo. Robert, do you have an idea?
Well, the comment atop SaveSlotToPath says: * This needn't actually be part of a checkpoint, but it's a convenient * location. And I agree with that. Incidentally, the wait-event handling in SaveSlotToPath() doesn't look right for the early-exit cases either. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
