Remove yield preparation and recovery logic from SBE stages

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Storage Execution
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Once we have the guarantee that a cursor's data will remain valid until the following call to next() we should remove the following:

      -The slotsAccessible() and disableSlotAccess() functions in sbe::PlanStage.
      -Any copies of data made as part of yield preparation (usually a call to makeOwned()).
      -Resetting cursor positions, as part of doSaveState() in leaf nodes (scan.cpp, ix_scan.cpp).
      -Tracking of _outerGetNext in the NLJ stage.

      This should be do-able in one patch. We will likely be able to remove doSaveState()/doRestoreState() from many of the sbe::PlanStages.

            Assignee:
            [DO NOT USE] Backlog - Storage Execution Team
            Reporter:
            Ian Boros
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: