Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

log4j-api: Incomplete/unclear description (simple implementation) #1474

Closed
Chealer opened this issue May 9, 2023 · 4 comments
Closed

log4j-api: Incomplete/unclear description (simple implementation) #1474

Chealer opened this issue May 9, 2023 · 4 comments
Labels
documentation Pull requests or issues that affect documentation

Comments

@Chealer
Copy link

Chealer commented May 9, 2023

log4j-api has the following name and description:

  <name>Apache Log4j API</name>
  <description>The Apache Log4j API</description>

This is at least incomplete, as shown by this discussion. Log4j developer Piotr P. Karwasz indicates that log4j-api provides a simple Log4j provider, SimpleLogger. I did not verify that and it is pretty much undocumented, but it is reportedly equivalent to slf4j-simple, except for being built in log4j-api.

I know little about Log4j, but considering that, I suggest replacing log4j-api's description with the following would make it much more complete/accurate:

Apache Log4j's logging facade API specification. A simple implementation (SimpleLogger) is built-in.

@rgoers
Copy link
Member

rgoers commented May 11, 2023

Why? I read the discussion and I doubt that the name and the description in the pom.xml are going to make any difference to anyone who cares.

What would really help is completely separating the documentation for Log4j-API from the documentation for the rest of Log4j.

@ppkarwasz ppkarwasz added the documentation Pull requests or issues that affect documentation label May 22, 2023
vy added a commit that referenced this issue Apr 26, 2024
@vy
Copy link
Member

vy commented Apr 26, 2024

@Chealer, thanks so much for the report. Fixed in 5a0262d. We will improve the situation further in:

@vy vy closed this as completed Apr 26, 2024
@Chealer
Copy link
Author

Chealer commented Apr 27, 2024

Thanks to you @vy
I commented the change and would suggest a small tuning:

This package contains 2 components of the Apache Log4j project's API, through which applications and libraries can log:

  • its specification
  • and its simple built-in implementation (SimpleLogger) for trivial use cases. A fully-fledged implementation, such as Log4j Core, should be preferred for production applications.

@vy
Copy link
Member

vy commented Apr 29, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Pull requests or issues that affect documentation
Projects
None yet
Development

No branches or pull requests

4 participants