Skip to content

Conversation

bigmontz
Copy link
Contributor

This scenario happens when the pull for a given address is purged while a connection stills in use and then another pool for the same address is created. In this case, the connection was being wrongly added to the existing pool.

The correct behaviour is destroy this orphan connections.

This scenario happens when the pull for a given address is purged while a connection stills in use and then another pool for the same address is created. In this case, the connection was being wrongly added to the existing pool.

The correct behaviour is destroy this orphan connections.
Copy link
Member

@robsdedude robsdedude left a comment

Choose a reason for hiding this comment

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

🌟

@bigmontz bigmontz force-pushed the 4.4-connections-which-pool-were-closed branch from 2172bb8 to d1a18a1 Compare December 17, 2021 14:25
@bigmontz bigmontz merged commit 24571eb into neo4j:4.4 Dec 17, 2021
@bigmontz bigmontz deleted the 4.4-connections-which-pool-were-closed branch December 17, 2021 15:46
bigmontz added a commit to bigmontz/neo4j-javascript-driver that referenced this pull request Dec 17, 2021
…4j#823)

This scenario happens when the pull for a given address is purged while a connection stills in use and then another pool for the same address is created. In this case, the connection was being wrongly added to the existing pool.

The correct behaviour is destroy this orphan connections.
bigmontz added a commit that referenced this pull request Dec 17, 2021
This scenario happens when the pull for a given address is purged while a connection stills in use and then another pool for the same address is created. In this case, the connection was being wrongly added to the existing pool.

The correct behaviour is destroy this orphan connections.
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