Skip to content

DNS change fix #817

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

Closed
wants to merge 1 commit into from
Closed

DNS change fix #817

wants to merge 1 commit into from

Conversation

injectives
Copy link
Contributor

This change fixes a problem when DNS name resolution changes to a different IP address and the driver fails to update its routing table. The existing connection pool implementation indexed connection pools by just hostnames and ports and ignored the resolved IP address, this update fixes it. We will also resolve the hostnames provided by the database immediately on cluster composition retrieval.

This change fixes a problem when DNS name resolution changes to a different IP address and the driver fails to update its routing table. The existing connection pool implementation indexed connection pools by just hostnames and ports and ignored the resolved IP address, this update fixes it. We will also resolve the hostnames provided by the database immediately on cluster composition retrieval.
@injectives injectives marked this pull request as draft February 3, 2021 22:17
@injectives
Copy link
Contributor Author

Closing in favour of #818

@injectives injectives closed this Feb 10, 2021
@injectives injectives deleted the bugfix/dns-change branch February 10, 2021 13:01
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.

1 participant