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

Refactor establishShardCursors() to return OwnedRemoteCursors

    XMLWordPrintableJSON

Details

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Major - P3 Major - P3
    • None
    • None
    • None
    • None
    • Query Execution

    Description

      Right now, establishShardCursors() returns RemoteCursors, which could cause cursors to leak if exceptions are thrown before callers have the chance to kill the cursors. The function should be modified to return OwnedRemoteCursors from the get-go so that no callers will need to worry about leaking cursors.

      Some callers place raw RemoteCursors into objects like AsyncResultsMerger, which should also be modified to take in owned cursors.

      Attachments

        Activity

          People

            backlog-query-execution Backlog - Query Execution
            davis.haupt@mongodb.com Davis Haupt (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated: