This will throw an EngineClosedException which is a different type and we do check for RecoveryEngineExceptions in some other place of the code. I'm not sure this matters, I think it's safer to keep as is? Also we loose the information about the recovery phase (1)