Improve docs for Persist::archive_persisted_channel
#3122
Labels
Take a Friday Leave a Friday
Stomp the Bugs, Without Much Commitment
In our current implementation of
Persist::archive_persisted_channel
we first read the monitor, write it at the archive location and finally lazily remove the monitor to 'move' it to the archive. This is fine, except that a lazy remove isn't guaranteed to finish if we encounter a persistence failure or happen to restart. This means that we could end up in a situation with the channel monitor at both, the original and the archive location until the call toarchive_fully_resolved_channel_monitors
succeeds once.All of this is fine, but I think we should clarify the docs:
Persist::archive_persisted_channel
docs state "Prevents the channel monitor from being loaded on startup.". We might want to clarify that this is not true if above mentioned scenario occurs.Persist::archive_persisted_channel
need to be idempotent, i.e., need to be able to be run multiple times without erroring if the monitor is already present at the archive location.The text was updated successfully, but these errors were encountered: