Demogorgon314 opened a new pull request #14945:
URL: https://github.com/apache/pulsar/pull/14945


   ### Motivation
   
   I'm working on improving the loadbalancer, but I find they are four classes 
used in loadbalancer and the package position is under the 
`org.apache.pulsar.broker`. This is strange because another class named 
`LocalBrokerData` is in `org.apache.pulsar.policies.data.loadbalancer`.
   
   We should move it to pulsar-common and put them together with 
`LocalBrokerData` for future extensions.
   
   ### Modifications
   
   Move loadbalancer used data classes package position to 
`org.apache.pulsar.policies.data.loadbalancer`
   
   ### Documentation
   
   Check the box below or label this PR directly (if you have committer 
privilege).
   
   Need to update docs? 
   
   - [ ] `doc-required` 
     
     (If you need help on updating docs, create a doc issue)
     
   - [x] `no-need-doc` 
     
     (Please explain why)
     
   - [ ] `doc` 
     
     (If this PR contains doc changes)
   
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to