Skip to content

Conn Pool management and load balancing strategy #39

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 6 commits into from
Oct 11, 2017

Conversation

zhenlineo
Copy link
Contributor

New sections in Config for introducing settings for connection pool management and load balancing strategy

New sections in Config for introducing settings for connection pool management and load balancing strategy
The driver maintains a pool of connections. The pooled connections are reused by sessions and transaction to avoid overhead added by establishing new connections every time to run a query.
The connections in the pool are created on demand. The connection pool always start with no pooled connection. New connections are created on demand by sessions and/or transactions. When sessions and/or transactions finished using connections, healthy ones are returned to the pool. These pooled connections are then reused by new sessions and transactions to execute queries.

Three connection pool related configuration settings are exposed via driver configuration, namely `maxConnectionLifetime`, `maxConnectionPoolSize` and `connectionAcquisitionTimeout`.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The name of the settings are following java parameter naming pattern, suggest to avoid java variables by using maximum connection lifetime, etc.

@zhenlineo
Copy link
Contributor Author

zhenlineo commented Oct 2, 2017

@zhenlineo zhenlineo requested a review from mariascharin October 2, 2017 15:44
@mariascharin
Copy link

Note that PR https://github.com/neo-technology/neo4j-manual-modeling/pull/346 must be merged at the same time as this one!

@mariascharin mariascharin merged commit 6fce47f into 1.5 Oct 11, 2017
@mariascharin mariascharin deleted the zhenlineo-1.5-new branch October 11, 2017 14:38
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