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

Address CSOT Compliance Issue in for Timeout Handling in Cursor Constructors

    • Type: Icon: Bug Bug
    • Resolution: Unresolved
    • Priority: Icon: Unknown Unknown
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Go Drivers
    • 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?

      Detailed steps to reproduce the problem?

      The CSOTĀ specifications states the following:

      If set, drivers MUST apply the timeoutMS option to the initial aggregate operation. Drivers MUST also apply the original timeoutMS value to each next call on the change stream but MUST NOT use it to derive a maxTimeMS field for getMore commands

      One can verify that the Go Driver does not do this by running the "timeoutMS applies to full resume attempt in a next call" CSOT test.

      Definition of done: what must be done to consider the task complete?

      The Go Driver team should consider updating our internal test runner to apply timeouts given to cursor constructors to be applied iteratively, as that is the expectation. Another option is to update unified spec tests / unified spec runner generally to satisfy this behavior. It's unclear if any other drivers have this problem, however.

      DRIVERS-2722 provides a long-term solution to this, but we should still consider a short-term solution. Most of the skipped tests are verifying other behavior and skipping them because we more maxTimeMS could hide bugs.

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

              Created:
              Updated: