Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DRIVERS-2091: remove references to getLastError #1698

Merged
merged 9 commits into from
Nov 11, 2024

Conversation

W-A-James
Copy link
Contributor

@W-A-James W-A-James commented Oct 29, 2024

Please complete the following before merging:

  • Update changelog.
  • Test changes in at least one language driver.
  • Test these changes against all server versions and topologies (including standalone, replica set, sharded
    clusters, and serverless).

@W-A-James W-A-James changed the title remove references to getLastError DRIVERS-2091: remove references to getLastError Oct 30, 2024
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: removed since everything in this document seemed to pertain to behaviour that was current in versions <= 2.6

Copy link
Member

Choose a reason for hiding this comment

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

I suggest we add this back and open a new drivers ticket to remove this spec.

@W-A-James W-A-James marked this pull request as ready for review October 30, 2024 22:33
@W-A-James W-A-James requested review from a team as code owners October 30, 2024 22:33
@W-A-James W-A-James requested review from nbbeeken and dariakp and removed request for a team October 30, 2024 22:33
Copy link
Member

Choose a reason for hiding this comment

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

I suggest we add this back and open a new drivers ticket to remove this spec.

3. On ServerVersion less than 2.6, drivers MUST NOT send a getLastError command for an Unacknowledged write concern.
4. FindAndModify helper methods MUST NOT send a write concern when the MaxWireVersion is less than 4.
5. Helper methods for other commands that write MUST NOT send a write concern when the MaxWireVersion is less than 5.
2. Commands supporting a write concern MUST send any non-default acknowledged write concern to the server in the
Copy link
Member

Choose a reason for hiding this comment

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

non-default acknowledged write concern -> non-default write concern. Unack'd writeConcerns need to be sent to the server too.

5. Helper methods for other commands that write MUST NOT send a write concern when the MaxWireVersion is less than 5.
2. Commands supporting a write concern MUST send any non-default acknowledged write concern to the server in the
command.
3. FindAndModify helper methods MUST NOT send a write concern when the MaxWireVersion is less than 4.
Copy link
Member

Choose a reason for hiding this comment

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

We can remove this since all drivers have dropped support for wire version <=4. 7 is the current min.

2. Commands supporting a write concern MUST send any non-default acknowledged write concern to the server in the
command.
3. FindAndModify helper methods MUST NOT send a write concern when the MaxWireVersion is less than 4.
4. Helper methods for other commands that write MUST NOT send a write concern when the MaxWireVersion is less than 5.
Copy link
Member

Choose a reason for hiding this comment

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

We can remove this for the same reason.

@W-A-James
Copy link
Contributor Author

Merging, will file ticket to remove driver-bulk-update spec

@W-A-James W-A-James merged commit ccb9816 into mongodb:master Nov 11, 2024
5 checks passed
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.

4 participants