Fixed a possible OutOfMemoryError and a NullPointerException - Githubissues

ryantenney / log4j-redis-appender

log4j appender writing straight to Redis

Other

33 stars 26 forks source link

Fixed a possible OutOfMemoryError and a NullPointerException #8

Open fhopf opened 8 years ago

fhopf commented 8 years ago

When running the appender in production we noticed two more problems

a NullPointerException when Redis ran full that could only be resolved by restarting the application
another cause for OutOfMemoryErrror: allowing to accept too many LogEvents

This pull request fixes both. For the first fix a bit of refactoring was involved to make the code somehow testable.