You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "onobc (via GitHub)" <gi...@apache.org> on 2023/02/10 14:03:33 UTC
[GitHub] [pulsar] onobc opened a new issue, #19484: [Bug] pulsar-client-all 2.11.0 transitively bringing in log4j2 implementation
onobc opened a new issue, #19484:
URL: https://github.com/apache/pulsar/issues/19484
### Search before asking
- [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.
### Version
- **OS:** `<not applicable>`
- **Pulsar:** `2.11.0`
### Minimal reproduce step
### Problem
The `pulsar-client-all:2.11.0` dependency transitively includes Log4j2 (see [ here](https://mvnrepository.com/artifact/org.apache.pulsar/pulsar-client-all/2.11.0)) rather than just the SLF4J api and logging bridge.
Whereas the `pulsar-client-all:2.10.3` does **NOT** bring a logging impl in (see [here](https://mvnrepository.com/artifact/org.apache.pulsar/pulsar-client-all/2.10.3)).
### Symptoms
If the consuming app is already using a logger impl then a conflict can occur at startup (when using SLF4J) telling user to pick one or the other.
If the consuming application is not already using a logger impl then `pulsar-client-all` effectively chooses LOG4J2 for the app.
### What did you expect to see?
Adding the dependency `org.apache.pulsar:pulsar-client-all:2.11.0` should **NOT** transitively bring in the log4j2 implementation.
### What did you see instead?
Adding the dependency `org.apache.pulsar:pulsar-client-all:2.11.0` transitively **DID** bring in the log4j2 implementation.
### Anything else?
_No response_
### Are you willing to submit a PR?
- [ ] I'm willing to submit a PR!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] tisonkun commented on issue #19484: [Bug] pulsar-client-all 2.11.0 transitively bringing in log4j2 implementation
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on issue #19484:
URL: https://github.com/apache/pulsar/issues/19484#issuecomment-1485173169
The pollute path is: pulsar-client -> pulsar-common -> bookkeeper:cpu-affinity -> bookkeeper:parent
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] github-actions[bot] commented on issue #19484: [Bug] pulsar-client-all 2.11.0 transitively bringing in log4j2 implementation
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #19484:
URL: https://github.com/apache/pulsar/issues/19484#issuecomment-1465393000
The issue had no activity for 30 days, mark with Stale label.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] tisonkun commented on issue #19484: [Bug] pulsar-client-all 2.11.0 transitively bringing in log4j2 implementation
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on issue #19484:
URL: https://github.com/apache/pulsar/issues/19484#issuecomment-1485076549
Thanks for reporting this issue. It seems we accidentally change log4j libs from test dep to compile dep. I'm looking into this one now.
cc @Technoboy- @codelipenghui @eolivelli possible related release manager
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] tisonkun commented on issue #19484: [Bug] pulsar-client-all 2.11.0 transitively bringing in log4j2 implementation
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on issue #19484:
URL: https://github.com/apache/pulsar/issues/19484#issuecomment-1485101091
This seems a bookkeeper issue https://github.com/apache/bookkeeper/commit/f0988ddf250440813866849cef4f734174a48fc4#r106236889.
We can force all log4j deps in test scope to workaround the case for now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] tisonkun commented on issue #19484: [Bug] pulsar-client-all 2.11.0 transitively bringing in log4j2 implementation
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on issue #19484:
URL: https://github.com/apache/pulsar/issues/19484#issuecomment-1486148389
Fixed and cherry-picked to 2.11 at 2d8af31007a94c8c126b77333188b8a98a2a37cf.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] tisonkun closed issue #19484: [Bug] pulsar-client-all 2.11.0 transitively bringing in log4j2 implementation
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun closed issue #19484: [Bug] pulsar-client-all 2.11.0 transitively bringing in log4j2 implementation
URL: https://github.com/apache/pulsar/issues/19484
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org