Yeah, this looks like a bug to me. Given that @slfritchie is fighting all sorts of delete edge cases related to data resurrection, even involving a new type of tombstone for bitcask, I think he would agree not to go ahead with the keydir delete here if the put fails. Let's see what he says.