Skip to content

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

Merged
merged 3 commits into from
Mar 4, 2025

Conversation

norareidy
Copy link
Collaborator

@norareidy norareidy commented Mar 3, 2025

JIRA - https://jira.mongodb.org/browse/DOCSP-47044

Copied from the PyMongo section on DNS SRV records

Copy link
Collaborator

@mongoKart mongoKart left a 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!

@@ -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
Copy link
Collaborator

Choose a reason for hiding this comment

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

no hyphen here AFAICT

Suggested change
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

Comment on lines 2 to 3
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
Copy link
Collaborator

Choose a reason for hiding this comment

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

s:

Suggested change
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

Comment on lines 3 to 4
the SRV connection format, the {+driver-short+} automatically re-scans for new hosts without
having to change the client configuration.
Copy link
Collaborator

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?

Suggested change
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.

Copy link
Collaborator Author

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

@norareidy norareidy requested a review from mongoKart March 3, 2025 21:01
Copy link
Collaborator

@mongoKart mongoKart left a comment

Choose a reason for hiding this comment

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

LGTM

@norareidy norareidy merged commit e118b05 into 10gen:main Mar 4, 2025
@norareidy norareidy deleted the DOCSP-47044-polling-srv branch March 4, 2025 14:54
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.

2 participants