Inside ReplicationCoordinatorImpl we currently invariant if we're going to deadlock the server.

However, this operation is quite aggressive and can cause general unavailability issues in case we find a problem. invariants should be reserved for dangerous unrecoverable situations and as such we should strive to replace them with tasserts if possible.