Uploaded image for project: 'PHP Driver: Library'
  1. PHP Driver: Library
  2. PHPLIB-808

Leave fullDocument change stream option unset by default

    • Type: Icon: Improvement Improvement
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 1.13.0-beta1, 1.13.0
    • Affects Version/s: None
    • Component/s: Spec Comp
    • Labels:
      None
    • Hide

      DRIVERS-2231:
      Currently, the ChangeStream spec defines the fullDocument field in ChangeStreamOptions as a String that defaults to the string literal "default". As of server version 3.6, sending "default" and sending no fullDocument field have identical behavior, so fullDocument should be defined as an Optional<String> to avoid using the "default" string literal.

      Drivers should sync legacy change streams tests to mongodb/specifications@31edce6 and legacy retryable reads tests to mongodb/specifications@006a4f1 .

      Show
      DRIVERS-2231 : Currently, the ChangeStream spec defines the fullDocument field in ChangeStreamOptions as a String that defaults to the string literal "default". As of server version 3.6, sending "default" and sending no fullDocument field have identical behavior, so fullDocument should be defined as an Optional<String> to avoid using the "default" string literal. Drivers should sync legacy change streams tests to mongodb/specifications@31edce6 and legacy retryable reads tests to mongodb/specifications@006a4f1 .

      This ticket was split from DRIVERS-2231, please see that ticket for a detailed description.

      Note: the Watch::FULL_DOCUMENT_DEFAULT constant will need to remain in place for BC, but can eventually be removed in the next major version (PHPLIB-818).

            Assignee:
            aleksandr.rudo@mongodb.com Aleksandr Rudo (Inactive)
            Reporter:
            dbeng-pm-bot PM Bot
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: