search

ryantenney / log4j-redis-appender

log4j appender writing straight to Redis
Other
33 stars 26 forks source link

re-worked appender to use jedis connection pooling and newer version. Th... #3

Closed rvanderwerf closed 8 years ago

rvanderwerf commented 9 years ago

I re-worked the code to use the newer jedis plugin, and use the JedisPool implementation for connection management. I added some robustness to connection testing/quality. It seems

This is to avoid the problem where the application thinks it is connected but is not actually connected. ( I am seeing this a lot). Apparently connection.isConnected() is not that good of a check if it's actually connected.

fhopf commented 8 years ago

It would be great if this pull request could be merged. We are running it in production with some further fixes that are in this pull request: https://github.com/rvanderwerf/log4j-redis-appender/pull/1

majcher commented 8 years ago

Could you release 1.0.2 please?