Avoid unnecessary shard version metadata refreshes

We're missing a `return Status::OK()` here that leads us to do refreshes even when we could be short-circuiting early.