[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: |
|
||||
| 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 ( |
| 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. |