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

Permit Versioned API options for getMore and transaction-continuing commands

    • Type: Icon: Improvement Improvement
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 5.0.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • Labels:
      None
    • Fully Compatible
    • Repl 2021-05-17

      The Versioned API design requires drivers and users to omit Versioned API parameters in the following two instances:
      1. When calling getMore
      2. In any command that is part of a transaction, except for the command starting the transaction.

      The reason for not allowing them on getMore calls is that it's bound to the API Version declared when creating the cursor. For transactions, the entire transaction operates under the same declared API Version, so later commands should not be able to override this.

      While this approach makes sense, it creates unnecessary workload in drivers. Drivers have to inspect command documents to know whether they should append Versioned API options for this instance. This can have a performance impact for drivers that already have raw BSON documents at this point.

      To fix this, Drivers and users should be required to include API versioning options for getMore and transaction-continuing commands, with the server emitting a command error if there is an API Version mismatch (exact comparison, e.g. apiVersion, apiStrict, and apiDeprecationErrors match).

      To aid with the transition to this new requirement, the options should be optional at first and can become required options once all drivers have updated their logic to always include these options.

            Assignee:
            jesse@mongodb.com A. Jesse Jiryu Davis
            Reporter:
            andreas.braun@mongodb.com Andreas Braun
            Votes:
            0 Vote for this issue
            Watchers:
            11 Start watching this issue

              Created:
              Updated:
              Resolved: