-
Type: Task
-
Resolution: Fixed
-
Priority: Major - P3
-
Affects Version/s: None
-
Component/s: None
-
None
-
Fully Compatible
-
Server Serverless 2022-07-11
Change the BasicCommandReplyBuilderInterface::invocation class to hold a DatabaseName object instead of a string for the db name. When constructing the db name member, we can grab the validated tenant from the OpMsg. Also change the following functions to take in DatabaseName rather than string as well -
- BasicCommandReplyBuilderInterface::runWithReplyBuilder
- BasicCommandReplyBuilderInterface::runAsync
- BasicCommandWithRequestParser::runWithReplyBuilder
- BasicCommand::runWithReplyBuilder
BasicCommandWithRequestParser::runWithReplyBuilder and BasicCommand::runWithReplyBuilder call other run() functions that take in a db name as a string - we'll change these in future tickets, so for now can just pass DatabaseName::toStringWithTenantId() to them.
We'll need to change the overridden runWithReplyBuilder() functions for any commands that inherit directly from BasicCommandReplyBuilderInterface - the HelloCmd and ClusterHelloCmd should be the only two.
- is duplicated by
-
SERVER-67356 Change BasicCommandWithReplyBuilderInterface::Invocation::ns() to generate namespace string with validated tenant id
- Closed
-
SERVER-67397 Change BasicCommandWithReplyBuilderInterface to use DatabaseName
- Closed
- related to
-
SERVER-67842 Complete TODO listed in SERVER-66561
- Closed