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

Fix ReadConcern logic on RunCommand

    • Type: Icon: Task Task
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 2.0.0, 1.13.0, 1.13.1
    • Affects Version/s: None
    • Component/s: None
    • None
    • Major Change
    • Not Needed
    • 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?

      readConcern is not currently passed to the underlying operation in command.go. This means that ReadConcern is effectively a no-op, and when we try to pass the Database read concern, here, it is not actually inherited by the operation. We should pass readConcern to the underlying operation, but this would be backward-breaking, so we should wait until the next major release of the driver to fix this small bug.

      Open questions:

      • Should we actually implement the readConcern logic or should we remove the ReadConcern option?

            Assignee:
            preston.vasquez@mongodb.com Preston Vasquez
            Reporter:
            benji.rewis@mongodb.com Benji Rewis (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: