Skip to content

Conversation

baileympearson
Copy link
Contributor

@baileympearson baileympearson commented Sep 2, 2025

Description

What is changing?

With these changes, the driver only throws errors when an unacknowledged write is used with a hint when the server does not support hint. This occurs in the following scenarios:

  • update commands: never (this is supported 4.2+, 4.2 is our minimum supported server version)
  • findAndModify and delete commands: 4.4+
Is there new documentation needed for these changes?

What is the motivation for this change?

Release Highlight

Hint is supported with unacknowledged writes for delete, update and findAndModify commands on servers that support hint

The driver no longer throws errors when hint is provided to unacknowledged writes for delete, update and findAndModify commands in the following circumstances:

  • No error is thrown for update commands.
  • No errors are thrown for delete and findAndModify commands on servers >=4.4.

Double check the following

  • Ran npm run check:lint script
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@baileympearson baileympearson changed the title feat(NODE-7134): allow hint with unacknowledged writes feat(NODE-7134): allow hint with unacknowledged writes for delete, update and findAndModify commands Sep 2, 2025
@baileympearson baileympearson marked this pull request as ready for review September 2, 2025 20:47
@baileympearson baileympearson requested a review from a team as a code owner September 2, 2025 20:47
@dariakp dariakp self-assigned this Sep 3, 2025
@dariakp dariakp added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Sep 3, 2025
@dariakp dariakp merged commit 82d6ce6 into mongodb:main Sep 5, 2025
28 of 31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Primary Review In Review with primary reviewer, not yet ready for team's eyes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants