log4j2 is misleading #4

Open
grobmeier opened this Issue Jan 6, 2013 · 3 comments

Comments

Projects
None yet
2 participants
Contributor

grobmeier commented Jan 6, 2013

log4j2 is short for log4j 2.0:
logging.apache.org/log4j/2.x

as it seems your appender is targeting log4j 1.x the name might be misleading.

Suggestion: update the readme file and point out for which version of log4j this appender will work.

Collaborator

leandrosilva commented Jan 6, 2013

No, it stands for Log4J To Redis, which means logs generated by Log4J is going to be stored on Redis.

Contributor

grobmeier commented Jan 6, 2013

Sure. But which version of log4j? log4j1.x or log4j2.x?
You can only find out that information by looking into the source code.

On Sun, Jan 6, 2013 at 3:14 PM, Leandro Silva notifications@github.comwrote:

No, it stands for Log4J To Redis, which means logs generated by Log4J is
going to be stored on Redis.


Reply to this email directly or view it on GitHubhttps://github.com/pavlobaron/log4j2redis/issues/4#issuecomment-11928893.

http://www.grobmeier.de
https://www.timeandbill.de

Collaborator

leandrosilva commented Jan 6, 2013

Oh, I got it now! :) So feel free to fork it and add that. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment