-
Notifications
You must be signed in to change notification settings - Fork 47
DOCSP-47044: Polling DNS SRV records #93
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.
a few small changes. let me know your thoughts!
dbx/srv-polling.rst
Outdated
@@ -0,0 +1,11 @@ | |||
To use DNS service discovery to look up the DNS SRV record of the service you're connecting to, | |||
specify the SRV connection format in your connection string. Additionally, if you enable | |||
the SRV connection format, the {+driver-short+} automatically re-scans for new hosts without |
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.
the SRV connection format, the {+driver-short+} automatically re-scans for new hosts without | |
the SRV connection format, the {+driver-short+} automatically rescans for new hosts without |
dbx/srv-polling.rst
Outdated
specify the SRV connection format in your connection string. Additionally, if you enable | ||
the SRV connection format, the {+driver-short+} automatically re-scans for new hosts without |
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:
specify the SRV connection format in your connection string. Additionally, if you enable | |
the SRV connection format, the {+driver-short+} automatically re-scans for new hosts without | |
specify the SRV connection format in your connection string. If you specify | |
this format, the {+driver-short+} automatically re-scans for new hosts without |
dbx/srv-polling.rst
Outdated
the SRV connection format, the {+driver-short+} automatically re-scans for new hosts without | ||
having to change the client 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.
this syntax is a little odd. it makes it sound like the driver (subject of the sentence) is the thing that has to or doesn't have to change the config, even though it's the user who does that. what if we just ditch the last part?
the SRV connection format, the {+driver-short+} automatically re-scans for new hosts without | |
having to change the client configuration. | |
the SRV connection format, the {+driver-short+} automatically re-scans for new hosts. |
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.
I added an extra sentence that's hopefully more clear, let me know what you think
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
JIRA - https://jira.mongodb.org/browse/DOCSP-47044
Copied from the PyMongo section on DNS SRV records