Skip to content

[release] Add redis.pool.testOnBorrow option to lib-jedis-pool — v1.1.0#95

Merged
pditommaso merged 1 commit into
masterfrom
add-jedis-pool-test-on-borrow
Jul 21, 2026
Merged

[release] Add redis.pool.testOnBorrow option to lib-jedis-pool — v1.1.0#95
pditommaso merged 1 commit into
masterfrom
add-jedis-pool-test-on-borrow

Conversation

@pditommaso

Copy link
Copy Markdown
Contributor

What & why

Adds a redis.pool.testOnBorrow option to lib-jedis-pool's JedisPoolFactory. When enabled, Jedis PING-validates each connection on borrow, so a connection left RESP-desynced but not broken by a prior use (a reply went unread, but the socket didn't break, so Jedis kept it is evicted instead of served to the next caller.

Without this, a single desynced connection poisons every subsequent borrower for its lifetime — the pool-wide containment discussed in libseqera#92 and platform#11820.

Change

  • maps →
  • Parametrized test asserting the flag reaches the pool config
  • README, changelog, VERSION → 1.1.0

Compatibility

  • Behavior: unchanged by default ( defaults to ).
  • API: the DI method gains one positional param. It's invoked by Micronaut via injection, not called directly, and there are no direct callers in platform or wave — so no consumer breaks. Bumped as a minor (1.0.0 → 1.1.0).

🤖 Generated with Claude Code
EOF
)

Enables PING-validation of pooled connections on borrow so a
RESP-desynced-but-not-broken connection is evicted instead of served to
the next caller. Defaults to false (existing behavior).

Backs platform#11820 / libseqera#92.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@pditommaso
pditommaso requested a review from jordeu July 21, 2026 15:13
@pditommaso
pditommaso merged commit 186535a into master Jul 21, 2026
3 checks passed
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