[SERVER-80128] Make ReplicationCoordinator::runCmdOnPrimaryAndAwaitResponse use DatabaseName obj Created: 16/Aug/23 Updated: 18/Aug/23 Resolved: 18/Aug/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Sophia Tan | Assignee: | [DO NOT USE] Backlog - Server Serverless (Inactive) |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | ntdi_must_have | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Assigned Teams: |
Serverless
|
||||||||||||
| Participants: | |||||||||||||
| Description |
|
The `ReplicationCoordinator::runCmdOnPrimaryAndAwaitResponse` is using `std::string& dbName`. This code works correctly as all call sites pass in global dbs instead of tenant dbs. We do not need to worry. But, as we decide to pass db name with `DatabaseName` across the code base, we need to change this function to use `DatabaseName` object.
|
| Comments |
| Comment by Didier Nadeau [ 18/Aug/23 ] |
|
This change was done as part of |