Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-89160

Refactor serializeToCommandObj() to serialize query settings via ExpCtx as serializeToCommandDoc()

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Major - P3 Major - P3
    • None
    • Affects Version/s: 8.0.0-rc0
    • Component/s: None
    • Query Execution

      serializeToCommandDoc() takes ExpCtx apart from AggRequest in order to serialize query settings into request body.

      serializeToCommandObj() is currently not doing that. This was not critical for propagating query settings as part of relevant aggregation requests, as this method seems to be used only for change streams, which do not apply query settings.

      This is not an issue per se, but an improvement. We need to either keep these methods consistent or keep only one or make one method use another.

            Assignee:
            Unassigned Unassigned
            Reporter:
            denis.grebennicov@mongodb.com Denis Grebennicov
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: