Addess recent performance regressions in mixed workloads (SERVER-74848)

[SERVER-74849] CommandInvocation: Don't re-parse NamespaceString and copy it repeatedly Created: 14/Mar/23  Updated: 29/Oct/23  Resolved: 16/Mar/23

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: 7.0.0-rc0

Type: Sub-task Priority: Major - P3
Reporter: Colin Stolley Assignee: Colin Stolley
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Related
related to SERVER-75529 Revisit memory layout for TenantId, D... Closed
Backwards Compatibility: Fully Compatible
Participants:

 Description   

CommandInvocation::ns() parses the NamespaceString from the request over and over again even though the dbName and request are both const. We should parse it once upon initialization and never again.

We can also be better about making copies of the NamespaceString, so do that too.



 Comments   
Comment by Githook User [ 16/Mar/23 ]

Author:

{'name': 'Colin Stolley', 'email': 'colin.stolley@mongodb.com', 'username': 'ccstolley'}

Message: SERVER-74849: Don't re-parse NamespaceString and copy it repeatedly
Branch: master
https://github.com/mongodb/mongo/commit/1f2032af64b4fcaaf62336df948999f7dbdd0b00

Generated at Thu Feb 08 06:28:42 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.