[SERVER-27222] Use NamespaceString consistently in CRUD path Created: 30/Nov/16  Updated: 06/Dec/22  Resolved: 23/Aug/18

Status: Closed
Project: Core Server
Component/s: Querying
Affects Version/s: None
Fix Version/s: None

Type: Improvement Priority: Major - P3
Reporter: James Wahlin Assignee: Backlog - Query Team (Inactive)
Resolution: Won't Fix Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
Assigned Teams:
Query
Participants:
Linked BF Score: 0

 Description   

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)



 Comments   
Comment by Ian Boros [ 23/Aug/18 ]

Closing as "won't fix" because this would be a large amount of mechanical work with no clear payoff. It would be more actionable if we targeted this change on specific hot code paths.

Generated at Thu Feb 08 04:14:31 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.