Skip to content

DRIVERS-3278 Revert field presence changes from server 8.2.0#1913

Open
abr-egn wants to merge 3 commits intomongodb:masterfrom
abr-egn:DRIVERS-3278/changestream-expanded-revert
Open

DRIVERS-3278 Revert field presence changes from server 8.2.0#1913
abr-egn wants to merge 3 commits intomongodb:masterfrom
abr-egn:DRIVERS-3278/changestream-expanded-revert

Conversation

@abr-egn
Copy link
Copy Markdown
Contributor

@abr-egn abr-egn commented Mar 18, 2026

  • [✔️] Is the relevant DRIVERS ticket in the PR title?
  • [✔️] Update changelog.
  • [✔️] Test changes in at least one language driver.
  • [✔️] Test these changes against all server versions and topologies (including standalone, replica set, and sharded
    clusters).

Tested on the Rust driver.

@abr-egn abr-egn changed the title Drivers 3278/changestream expanded revert DRIVERS-3278 Revert field presence changes from server 8.2.0 Mar 18, 2026
@abr-egn abr-egn marked this pull request as ready for review March 18, 2026 11:01
@abr-egn abr-egn requested a review from a team as a code owner March 18, 2026 11:01
@abr-egn abr-egn requested a review from BorisDog March 18, 2026 11:01
@@ -139,10 +139,10 @@

/**
* Only present for ops of type 'rename', 'create', 'modify', 'createIndexes', 'dropIndexes', 'shardCollection', 'reshardCollection', 'refineCollectionShardKey'.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

optional: I suggest to be consistent with the formatting of collectionUUID. i.e. having each operation on a separate row.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

*
* @since 6.0.0
*/
operationDescription: Optional<Document>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this change apply to operationDescription as well? I didn't see it mentioned in DRIVERS-3278.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does, the change reverts all of the fields that were exposed in SERVER-86688 (visible in the implementation PR). The lack of mention in DRIVERS-3278 was an oversight.

@abr-egn abr-egn requested a review from BorisDog April 2, 2026 16:23
@abr-egn abr-egn force-pushed the DRIVERS-3278/changestream-expanded-revert branch from 22343c8 to 28a5139 Compare April 2, 2026 16:27
Copy link
Copy Markdown
Contributor

@BorisDog BorisDog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test passes in C#.
LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants