DOCSP-4466 Add CSOT #951
Open
DOCSP-4466 Add CSOT #951
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Deploy Preview for docs-node ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
lindseymoore
requested changes
Dec 20, 2024
mongoKart
requested changes
Jan 2, 2025
mongoKart
approved these changes
Jan 7, 2025
lindseymoore
approved these changes
Jan 8, 2025
| Overview | ||
| -------- | ||
|
|
||
| When you use {+driver-short+} to perform a server operation, you can also limit |
There was a problem hiding this comment.
Suggested change
| When you use {+driver-short+} to perform a server operation, you can also limit | |
| When you use the {+driver-short+} to perform a server operation, you can also limit |
S: I think we normally put 'the' in front of the the driver name!
| specify a **client-side operation timeout (CSOT)**. The timeout applies to all steps | ||
| needed to complete the operation, including server selection, connection | ||
| checkout, serialization, and server-side execution. When the timeout expires, | ||
| {+driver-short+} raises a timeout exception. |
There was a problem hiding this comment.
Suggested change
| {+driver-short+} raises a timeout exception. | |
| the {+driver-short+} raises a timeout exception. |
the
| ~~~~~~~~~~~~~~~~~~~ | ||
|
|
||
| When you specify a ``timeoutMS`` option, the driver applies the timeout | ||
| according to the same inheritance behaviors as other {+driver-short+} options. |
There was a problem hiding this comment.
Suggested change
| according to the same inheritance behaviors as other {+driver-short+} options. | |
| according to the same inheritance behaviors as the other {+driver-short+} options. |
S: Could put 'the' here as well, since you're talking about a specific set of options mentioned previously
| ``getMore()`` method must finish within the limit specified by the ``timeoutMS`` | ||
| option. If they do not, the system throws a timeout error. | ||
|
|
||
| When you close a cursor by calling the ``toArray()``or ``close()`` method, the |
There was a problem hiding this comment.
Suggested change
| When you close a cursor by calling the ``toArray()``or ``close()`` method, the | |
| When you close a cursor by calling the ``toArray()`` or ``close()`` method, the |
I: The space fixes the formatting on the page!
Comment on lines
+208
to
+210
| As shown in the following example, the cursor continues to fetch new documents | ||
| as they are added to a collection, and then times out if it takes longer than 10 | ||
| seconds to retrieve documents: |
There was a problem hiding this comment.
Suggested change
| As shown in the following example, the cursor continues to fetch new documents | |
| as they are added to a collection, and then times out if it takes longer than 10 | |
| seconds to retrieve documents: | |
| As shown in the following example, the cursor continues to fetch new documents | |
| as they are added to a collection and times out if it takes longer than 10 | |
| seconds to retrieve documents: |
S: Don't think that second clause is independent, so I removed the comma and 'then'. Could also separate into two sentences.
Style Guide on 'then' : https://www.mongodb.com/docs/meta/style-guide/terminology/alphabetical-terms/#t
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-44666
Staging - https://deploy-preview-951--docs-node.netlify.app/fundamentals/connection/csot/
Self-Review Checklist