copying from a comment on the commit-   nit: now that we folded the close and performRecoveryRestart into `status.resetRecovery()` we don't need the success pattern anymore. This can be:  ``` if (onGoingRecoveries.replace(id, status, resetRecovery) == false) {        resetRecovery.cancel("replace failed");        throw new IllegalStateException("failed to replace recovery target"); }  ``` 