Uploaded image for project: 'Go Driver'
  1. Go Driver
  2. GODRIVER-2861

Merge driver.ListCollectionsBatchCursor into driver.BatchCursor

XMLWordPrintableJSON

    • Type: Icon: Improvement Improvement
    • Resolution: Duplicate
    • Priority: Icon: Unknown Unknown
    • None
    • Affects Version/s: None
    • Component/s: None
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?

      Both driver.ListCollectionsBatchCursor and mongo.Cursor implement the mongo.batchCursor interface and the driver.BatchCursor abstraction.

      driver.ListCollectionsBatchCursor was added in GODRIVER-1065 , the original commit on gerrithub can be found here. It is not immediately clear why drivers.ListCollectionsBatchCursor was decouple from mongo.Cursor, but it may be because the former had to support the < 3.0 legacy operation for listing a collection. The Go Driver no longer supports versions < 3.6 and so all of this logic can be deprecated.

      This ticket proposes to update the operation.ListCollection#Result to return driver.BatchCursor rather than driver.ListCollectionsBatchCursor.

      Definition of complete: the aforementioned change is created and any addition updates required for a green build.

            Assignee:
            Unassigned Unassigned
            Reporter:
            preston.vasquez@mongodb.com Preston Vasquez
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: