merlimat opened a new pull request #10616:
URL: https://github.com/apache/pulsar/pull/10616
### Motivation
`MessageCrypto` is included in `pulsar-client-api` and it's exposing Netty
`ByteBuf` in its API.
The problem is that we are shading Netty and the classes are renamed. We
must not have external libraries symbols as part of our API.
The result is that Netty symbols are exposed as
`org.apache.pulsar.shade.io.netty...` and therefore are not going to be
interoperable with other libraries.
Because of this, `pulsar-client-api` was also included into the shaded Jars
and that brings in other issues (like missing the sources/javadocs of the
PulsarClient APIs).
### Modifications
* Removed Netty dependency on `pulsar-client-api`
* Made breaking change in the MessageCrypto to use `java.nio.ByteBuffer` in
API
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
