new description

We've decided not to do what this ticket originally aimed to do. Remove TODO's for this ticket in the codebase.

old description
Passing an empty NamespaceString to OperationShardingState::initializeClientRoutingVersionsFromCommand()   means the OperationShardingState cannot correctly store the shard/database version tied to the command.

service_entry_point_common execCommandDatabase calls oss.initializeClientRoutingVersionsFromCommand(invocation->ns(), request.body).

However, not all commands that go through this path are derived from InvocationBase - thus invocation->ns() yields an empty NamespaceString. 

List of commands we found that pass an empty NamespaceString to OperationShardingState::initializeClientRoutingVersionsFromCommand() that aren't derived from InvocationBase:

• ConfigSvrMovePrimaryCommand
• ConfigSvrEnableShardingCommand

Additionally, DropConnectionsCmd derives from InvocationBase but passes an empty NamespaceString to OperationShardingState::initializeClientRoutingVersionsFromCommand().