Skip to content
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

Keep seed router address in the pool after routing table #931

Open
wants to merge 1 commit into
base: 4.4
Choose a base branch
from

Conversation

bigmontz
Copy link
Contributor

@bigmontz bigmontz commented May 2, 2022

Forgetting the seed router was causing the connection to the seed router being recreated whenever the routing table is fetched and the seed router is not part of the cluster formation. For cases where the database name is set in the session, the driver will have to fetch the routing table for the default/home database for discovering the database name and make it consitent during the session lifetime. In this scenario, the seed router will be always used during the discovery. So, a new connection to the seed router will be created for each session.

The solution for this issue is to keep the seed router in the pool during the update routing table process. This way, the same connections for the seed router can be re-used.

Forgetting the seed router was causing the connection to the seed router being recreated whenever the routing table is fetched and the seed router is not part of the cluster formation. For cases where the database name is set in the session, the driver will have to fetch the routing table for the default/home database for discovering the database name and make it consitent during the session lifetime. In this scenario, the seed router will be always used during the discovery. So, a new connection to the seed router will be created for each session.

The solution for this issue is to keep the seed router in the pool during the update routing table process. This way, the same connections for the seed router can be re-used.
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