-
Type: Task
-
Resolution: Fixed
-
Priority: Major - P3
-
Affects Version/s: None
-
Component/s: None
-
Serverless
-
Fully Compatible
-
Server Serverless 2023-06-12, Server Serverless 2023-06-26, Server Serverless 2023-07-10, Server Serverless 2023-07-24, Server Serverless 2023-08-07
-
3
Remove the NamespaceString ctors that take a default parameter. At the point we do this ticket, most (hopefully all!) places that need a tenantId should already be passed one. If there are any places that don't take in a TenantId explicitly yet, we'll need to decide whether we should pass boost::none explicitly, or whether there is an actual tenantId that should be passed.
Any unit tests can take in boost::none (unless they test multitenancy). If there are any places where an explicit tenantId should be passed that we haven't changed yet, we can change this as a part of this ticket if the tenantId we want to pass is available easily. If there is work to be done to thread the tenantId through, we should file a separate ticket to do that work, pass boost::none at the callsite, and leave a TODO on the filed ticket.
- depends on
-
SERVER-77689 Remove use of NamespaceString ctors with default parameter for tenantId in unit test
- Closed
-
SERVER-78926 Replace calls to deprecated NamespaceString constructor with default constructor
- Closed
-
SERVER-78928 Create constant NamespaceString instead of using deprecated NSS constructors
- Closed
-
SERVER-79313 Remove NamespaceStringOrUUID ctors with default parameter for tenantId
- Closed
- duplicates
-
SERVER-76853 Avoid recreating a NamespaceString object from ns string in logging
- Closed
- is depended on by
-
SERVER-65456 Make NamespaceString::db_deprecated() private
- Closed
- is related to
-
SERVER-78942 Replace `forTest` calls in NamespaceStringTest with constructor calls directly
- Closed