GitHub user jeantil added a comment to the discussion: pulsar admin client in 
pulsar-client-all logs a ClassNotFoundException at startup

Hello, 

Thanks for your answer, I realize I misunderstood and shading is not the root 
cause. Thanks for clarifying this. 

I would however argue that this log gives a bad developer experience to users 
of the pulsar client: 
- following the normal usage documentation of the client will result in the 
user's application throwing a full on warning with a `ClassNotFoundException` 
and a full stack trace. 
- the warning provides absolutely no indication on how to resolve the issue, 
not even an indication of why it would be important to use conscrypt instead of 
the JDK security provider
- the pulsar client documentation doesn't mention conscrypt, doesn't explain 
why users of the client should deploy it

So either it is important to use conscrypt, in which case it would be nice to 
know why or its not and this warning should be downgraded to an INFO log. 
In both cases, the full stacktrace and classcast exception message doesn't 
bring much useful information and could be logged at a lower level (probably 
DEBUG) 

would you be open to a pull request in to that effect? 

GitHub link: 
https://github.com/apache/pulsar/discussions/20057#discussioncomment-5754271

----
This is an automatically sent email for commits@pulsar.apache.org.
To unsubscribe, please send an email to: commits-unsubscr...@pulsar.apache.org

Reply via email to