Skip to content

fix(java/driver/flight-sql): cache closes primary connection and attempts to re-use it. - #4503

Closed
Kyperr wants to merge 1 commit into
apache:mainfrom
Kyperr:java-fix-primary-client-invalidation
Closed

Kyperr wants to merge 1 commit into
apache:mainfrom
Kyperr:java-fix-primary-client-invalidation

Conversation

@Kyperr

@Kyperr Kyperr commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

If the cache isn't touched for 5 minutes, it triggers a lazy-evct close of the primary connection, which, could be used by other things, or prevent this from being used in pools.

@Kyperr
Kyperr requested a review from lidavidm as a code owner July 10, 2026 18:40
@amoeba

amoeba commented Jul 10, 2026

Copy link
Copy Markdown
Member

Hi @Kyperr, can you please have a look at the failing CI here: https://github.com/apache/arrow-adbc/actions/runs/29115387188/job/86439165716?pr=4503 and address that? Also, there isn't any context here for the issue this PR is trying to solve. Is there an issue we could associate or do you have a minimal reproducer?

@Kyperr Kyperr changed the title Fixes an issue where the FlightSqlConnection becomes unusable fix(java/driver/flight-sql): close session on connection close to resolve session leaks Jul 13, 2026
@Kyperr Kyperr changed the title fix(java/driver/flight-sql): close session on connection close to resolve session leaks fix(java/driver/flight-sql): cache closes primary connection and attempts to re-use it. Jul 13, 2026
@Kyperr

Kyperr commented Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

Oh man. Sorry about that. I did not mean to open this yet.

@Kyperr Kyperr closed this Jul 13, 2026
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.

3 participants