so looking at the code again, i believe this was written to be defensible to "unpredictable" corner cases. The `Mod:delete` is not run if either the operation has already been marked as complete (in which case running it again would be fine) or if the vnode type is not what we expected (in which case we may have hit some weird case where we shouldn't delete the data now). Initially i thought to get rid of the case clause but I think now it should remain. I think its still safe to shutdown the pool and such because we will unregister the vnode and it can be restarted if needed but at least the data is still there. alternatively, could differentiate between the wrong vnode type case and the already marked complete case to be extra clear/safe.