Skip to content

Conversation

@injectives
Copy link
Contributor

Cherry-picks: #1027, #1031, #1034, #1035, #1037.

Routing table address set update may override resolved router address. This leads to routing connection pool closures. This update aims to optimise this.
This update introduces read/write lock for internal pool map access management. For instance, the `retainAll` method is executed with write lock.
* Replace AddressSet with standard Java collections

This update also prevents unnecessary connection pool closures by keeping `disused` addresses until next routing table update and reusing them.

* Update reader, writer and router addresses handling and change type
@injectives injectives changed the title Backport connection pool improvements, InternalSummaryCounters update and Testkit launch fix Backport connection pool improvements, routing table handling, InternalSummaryCounters update and Testkit launch fix Oct 29, 2021
@injectives injectives requested a review from gjmwoods October 29, 2021 09:41
@injectives injectives merged commit 7fa8c17 into neo4j:4.3 Oct 29, 2021
@injectives injectives deleted the feature/bp branch October 29, 2021 12:33
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