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

Move SearchSharedState off ExpressionContext

    • Type: Icon: Task Task
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 8.1.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • None
    • Query Integration
    • Fully Compatible

      1. Move SearchSharedState off ExpressionContext into DocumentSourceInternalSearchIdLookup header, and it might be worth considering renaming the struct since "SearchSharedState" is pretty vague. What about "SearchIdLookupMetrics"?
      2. IdLookup can construct its own instance of the struct, and then DocumentSourceInternalSearchMongotRemote needs a pointer to the struct. This Is the function where we "desugar" a $search stage into the separate $internalSearchMongotRemote and $_internalSearchIdLookup stages, and we'll need to pass the pointer from idLookup to mongotRemote. If we'reĀ _not using an idLookup stage (if the query is "storedSource"), then mongot remote can have a nullptr.
      3. Will need to move the test from the previous ticket form expression context to idLookup, but the integrity of the test should be able to remain.

            Assignee:
            joseph.shalabi@mongodb.com Joe Shalabi
            Reporter:
            will.buerger@mongodb.com Will Buerger
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: