redis / jedis

Redis Java client

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[Suggested change] Log a warning when `ConnectionFactory.validateObject` determines the connection is no longer connected and/or the ping fails

kdrakon opened this issue · comments

Expected behavior

When a connection is evicted in JedisPool, this is done quietly in the background.

Actual behavior

While it seems the eviction is working correctly—via the Apache commons-pool library—the Jedis library logs an ERROR level log when determining the connection is no longer valid.

Steps to reproduce:

To reproduce, you need to have your pool connected and simulate an 'invalid' connection by having the server drop the connection to the Jedis client pool.

Suggested change:

Because it is expected that the connection may no longer be valid, I suggest we log a WARN instead of an ERROR. I've made a small PR that makes that change here: #3747

Closing. See #3747