-
Type:
Task
-
Resolution: Unresolved
-
Priority:
Major - P3
-
None
-
Affects Version/s: None
-
Component/s: None
-
None
-
Server Programmability
-
None
-
3
-
TBD
-
None
-
None
-
None
-
None
-
None
-
None
-
None
strategy.cpp defines much of the common router command infrastructure. A significant amount of the code concerns generic command logic, and yet another significant portion of the code concerns routing. The file is currently owned by Server Programmability, which does not conceptually have the expertise or authority to make decisions about the routing model. At a minimum, the catalog-specific code should be owned by CAR and the code could probably benefit from more separation of concerns/domains.
Separate out the catalog-specific logic in strategy.cpp and mark CAR as the code owners of that logic. Alternatively, donate strategy.cpp in its entirety to CAR, although it seems appropriate for SP to own things like ParseAndRunCommand::_parseCommand(), which doesn't appear to involve anything specific to routing.
- related to
-
SERVER-103706 Get rid of the stale errors handling on strategy.cpp
-
- Closed
-