[SERVER-69673] Remove DatabaseName ctors with default parameter for tenantId Created: 14/Sep/22  Updated: 29/Oct/23  Resolved: 09/May/23

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

Type: Task Priority: Major - P3
Reporter: Janna Golden Assignee: Mathis Bessa
Resolution: Fixed Votes: 0
Labels: ntdi_must_have
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Assigned Teams:
Serverless
Backwards Compatibility: Fully Compatible
Sprint: Server Serverless 2023-05-15
Participants:
Story Points: 1

 Description   

Remove the DatabaseName 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.



 Comments   
Comment by Githook User [ 09/May/23 ]

Author:

{'name': 'mathisbessamdb', 'email': 'mathis.bessa@mongodb.com', 'username': 'mathisbessamdb'}

Message: SERVER-69673 removed parametric constructor with default tenant
Branch: master
https://github.com/mongodb/mongo/commit/93823e0104dc279cb42428e9ef2f4d79bb46a512

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