Now that we should wrap every "scatterGatter" function call with a CollectionRouter, we should not allow calling a "scatterGatter" function without throwing  on a stale ShardVersion error. This is because the CollectionRouter wrapper can only handle errors that are thrown.

Therefore, we should remove the function `scatterGatherVersionedTargetByRoutingTableNoThrowOnStaleShardVersionErrors`.

It's only being used in shardsvr_drop_indexes.cpp in order to skip those shards that already succeeded on a previous iteration. To keep that behavior, we can add the same functionality to the already existing function `scatterGatherVersionedTargetByRoutingTable`.