-
Notifications
You must be signed in to change notification settings - Fork 24
DOCSP-49056: Server Selection #618
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
DOCSP-49056: Server Selection #618
Conversation
✅ Deploy Preview for docs-csharp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Nice work! Just a few suggestions
when you don't specify any custom server-selection logic, see |
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:
To learn more about the default server-selection algorithm, which the driver follows | |
when you don't specify any custom server-selection logic, see | |
The driver follows a default server-selection algorithm when you don't specify any custom server-selection logic. To learn more about the default server-selection algorithm, see |
* - ``ReadPreference`` | ||
- | The client's default read-preference settings. ``MaxStaleness`` represents the | ||
| longest replication lag (in wall-clock time) that a secondary can experience and |
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.
Is wall-clock time a standard term? I've never heard that before
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.
Looks like it's one of many ways to refer to real time. Can change it to that instead.
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!
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
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-49056
Staging Links
Self-Review Checklist