search

kmtong / logback-redis-appender

Logback Redis Appender
Apache License 2.0
114 stars 64 forks source link

RedisAppender should be asynchronous #13

Closed GuiSim closed 9 years ago

GuiSim commented 9 years ago

Currently, logs written to the RedisAppender are sent using the same thread as the one that called the logger.

This can lead to issues if the Redis server is down, if there is unexpected network latency or any other delay in communication.

If there are no plans to change this behavior, I suggest that the documentation clearly states this. The documentation should also refer users to logback's AsyncAppender and highly recommend its usage.

Thanks!

kmtong commented 9 years ago

Thanks for the comments. I would look into it once I have time.

kmtong commented 9 years ago

AsyncAppender is cool. Included some hints in the README. Thanks.