Currently the CRUD command path uses a mix of NamespaceString and std::string to represent namespaces and can construct objects representing the same namespace multiple times over a commands lifetime. We should make use of NamespaceString (replacing std::string) consistently and pass rather than reconstruct where possible to avoid the cost of string creation.

(A similar request (SERVER-18137) is in place for sharding)