Skip to content

Conversation

injectives
Copy link
Contributor

@injectives injectives commented Aug 27, 2025

BREAKING CHANGE: org.neo4j.driver.Config.ConfigBuilder#withConnectionAcquisitionTimeout(long, TimeUnit) no longer accepts negative values, matching the handling of org.neo4j.driver.Config.ConfigBuilder#withConnectionTimeout(long, TimeUnit). In addition, the acquisition timeout is applied for the entire connection acquisition process that may include client-side routing and other logic. Previously, the connection acquisition timeout applied individually per every acquisition from individual connection pool and only when all connections were busy.

@injectives injectives force-pushed the feature/acquisition branch 7 times, most recently from e5b59f0 to ad13ed8 Compare August 28, 2025 16:15
@injectives injectives changed the title update feat(acquisition): apply acquisition timeout immediately Aug 28, 2025
@injectives injectives force-pushed the feature/acquisition branch 6 times, most recently from bf18841 to 629225e Compare September 1, 2025 17:22
@injectives injectives changed the title feat(acquisition): apply acquisition timeout immediately feat(acquisition): apply acquisition timeout to all steps Sep 1, 2025
BREAKING CHANGE: `org.neo4j.driver.Config.ConfigBuilder#withConnectionAcquisitionTimeout(long, TimeUnit)` no longer accepts negative values, matching the handling of `org.neo4j.driver.Config.ConfigBuilder#withConnectionTimeout(long, TimeUnit)`. In addition, the acquisition timeout is applied for the entire connection acquisition process that may include client-side routing and other logic. Previously, the connection acquisition timeout applied individually per every acquisition from individual connection pool and only when all connections were busy.
Copy link

@MaxAake MaxAake left a comment

Choose a reason for hiding this comment

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

✅ Looks good!

@injectives injectives merged commit f3a8572 into neo4j:6.x Sep 2, 2025
10 of 24 checks passed
@injectives injectives deleted the feature/acquisition branch September 2, 2025 08:35
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