Several methods in ReplicationCoordinatorImpl invariant on (isRSTLLocked() || isLockFreeReadsOp()). The first check is a bit more expensive since we take a spinlock and perform a map lookup, whereas the second just checks an int. We should swap around the checks.