freeznet opened a new pull request #9842:
URL: https://github.com/apache/pulsar/pull/9842


   Fixes #9640
   
   ### Motivation
   
   Module `pulsar-client-admin-api` has been introduced in #9246, but it 
pulling in many dependencies.  That is undesirable for an "API" module. As 
@sijie @merlimat and @jerrypeng suggests, this pr resolves the issue and make 
`pulsar-client-admin-api` has minimal transient dependences.
   
   ### Modifications
   
   - [ ] pulsar-client-admin-api should only depend on pulsar-client-api
   - [ ] Move pojos from pulsar-common to pulsar-client-admin-api module
   - [ ] Use lombok to expose fields in pojos with autogenerated getters
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   - [ ] Check  `pulsar-client-admin-api` with minimal transient dependences.
     
   ```
   $ mvn -pl pulsar-client-admin-api dependency:tree -Dscope=runtime
   [INFO] --- maven-dependency-plugin:3.1.2:tree (default-cli) @ 
pulsar-client-admin-api ---
   [INFO] org.apache.pulsar:pulsar-client-admin-api:jar:2.8.0-SNAPSHOT
   [INFO] \- org.apache.pulsar:pulsar-client-api:jar:2.8.0-SNAPSHOT:compile
   [INFO] 
------------------------------------------------------------------------
   ```
   


----------------------------------------------------------------
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]


Reply via email to