You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/04/04 13:28:32 UTC

[GitHub] [pulsar] Demogorgon314 opened a new pull request, #14945: [refactor][broker] Move loadbalancer used data classes package position

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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] RobertIndie merged pull request #14945: [refactor][broker] Move loadbalancer used data classes package position

Posted by GitBox <gi...@apache.org>.
RobertIndie merged PR #14945:
URL: https://github.com/apache/pulsar/pull/14945


-- 
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] Demogorgon314 closed pull request #14945: [refactor][broker] Move loadbalancer used data classes package position

Posted by GitBox <gi...@apache.org>.
Demogorgon314 closed pull request #14945: [refactor][broker] Move loadbalancer used data classes package position
URL: https://github.com/apache/pulsar/pull/14945


-- 
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] Demogorgon314 commented on pull request #14945: [refactor][broker] Move loadbalancer used data classes package position

Posted by GitBox <gi...@apache.org>.
Demogorgon314 commented on PR #14945:
URL: https://github.com/apache/pulsar/pull/14945#issuecomment-1090973022

   @codelipenghui @RobertIndie The CI is passed. Please take a look if you have time, thanks : )


-- 
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] Demogorgon314 closed pull request #14945: [refactor][broker] Move loadbalancer used data classes package position

Posted by GitBox <gi...@apache.org>.
Demogorgon314 closed pull request #14945: [refactor][broker] Move loadbalancer used data classes package position
URL: https://github.com/apache/pulsar/pull/14945


-- 
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] Demogorgon314 commented on pull request #14945: [refactor][broker] Move loadbalancer used data classes package position

Posted by GitBox <gi...@apache.org>.
Demogorgon314 commented on PR #14945:
URL: https://github.com/apache/pulsar/pull/14945#issuecomment-1087044680

   /pulsarbot rerun-failure-checks


-- 
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] Demogorgon314 commented on pull request #14945: [refactor][broker] Move loadbalancer used data classes package position

Posted by GitBox <gi...@apache.org>.
Demogorgon314 commented on PR #14945:
URL: https://github.com/apache/pulsar/pull/14945#issuecomment-1089980899

   /pulsarbot rerun-failure-checks


-- 
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] Demogorgon314 commented on pull request #14945: [refactor][broker] Move loadbalancer used data classes package position

Posted by GitBox <gi...@apache.org>.
Demogorgon314 commented on PR #14945:
URL: https://github.com/apache/pulsar/pull/14945#issuecomment-1089880353

   /pulsarbot rerun-failure-checks


-- 
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] Demogorgon314 closed pull request #14945: [refactor][broker] Move loadbalancer used data classes package position

Posted by GitBox <gi...@apache.org>.
Demogorgon314 closed pull request #14945: [refactor][broker] Move loadbalancer used data classes package position
URL: https://github.com/apache/pulsar/pull/14945


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