-
Type: Improvement
-
Resolution: Fixed
-
Priority: Major - P3
-
Affects Version/s: None
-
Component/s: None
-
None
-
Service Arch
-
Fully Compatible
-
Networking & Obs 2024-07-08, Networking & Obs 2024-07-22, Networking & Obs 2024-08-05, Networking & Obs 2024-08-19, N&O Needs Merge
ClusterFindCmdBase::parse does not actually parse the command request, and instead that work is deferred until the command invocation is run, which is unexpected. Deferring parsing will have also a performance cost once SERVER-85791 is merged, since the generic arguments need to be parsed in parse, so deferring will result in them being parsed twice.
We should update ClusterFindCmd's implementation to actually parse the request in its parse method. The shard's implementation of find has already been updated to do this in SERVER-86084, so this will also make the two command implementations more consistent.
- related to
-
SERVER-85791 Only parse command request once in command processing path
- Closed
-
SERVER-86084 Ensure find command parsing occurs in FindCmd::parse
- Closed