-
Notifications
You must be signed in to change notification settings - Fork 33
DOCSP-42256: leftover v2 migration items #377
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM w/ a question and suggestion
source/whats-new.txt
Outdated
``Cursor.SetMaxAwaitTime()``. This method specifies the maximum time | ||
that the server waits for new documents when you retrieve documents by | ||
using a tailable cursor when querying a capped collection. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
S: This is a pretty wordy sentence. Can it be simplified to something like:
``Cursor.SetMaxAwaitTime()``. This method specifies the maximum time | |
that the server waits for new documents when you retrieve documents by | |
using a tailable cursor when querying a capped collection. | |
``Cursor.SetMaxAwaitTime()``. This method specifies the maximum time | |
that the server waits for new documents when retrieving documents with a tailable cursor on a capped collection. |
source/whats-new.txt
Outdated
@@ -122,6 +122,13 @@ The 2.0 {+driver-short+} release includes the following improvements and fixes: | |||
``SessionOptions`` type. To learn more, see the | |||
:ref:`golang-session-txn-options` section of the Transactions guide. | |||
|
|||
- The ``Cursor.SetMaxTime()`` method is renamed to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Q: Is this a breaking change since it changes a method name?
@prestonvasquez reminder for you to review! |
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-42256
Staging - https://preview-mongodbrustagir.gatsbyjs.io/golang/DOCSP-42256-v2-mig-items/whats-new/
Self-Review Checklist