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/06/02 15:46:37 UTC

[GitHub] [pulsar] frankjkelly opened a new issue, #15904: [Doc] Load Balancer - Broker overload thresholds does not apply to some Shedding algorithms

frankjkelly opened a new issue, #15904:
URL: https://github.com/apache/pulsar/issues/15904

   ### What issue do you find in Pulsar docs?
   
   In talking with @sijie he mentioned that the `loadBalancerBrokerOverloadedThresholdPercentage` setting does not apply to the ThresholdShedder
   
   However if you look at the documentation for "Broker overload thresholds" that heading comes at the same level and right after "UniformLoadShedder"
   
   https://pulsar.apache.org/docs/next/administration-load-balance#broker-overload-thresholds
   
   So it reads as if `loadBalancerBrokerOverloadedThresholdPercentage` might be a setting use across all shedding algorithms?
   
   
   
   ### What is your suggestion?
   
   Clarify if `loadBalancerBrokerOverloadedThresholdPercentage` applies to some or all shedding algorithms and place that setting in the appropriate section
   
   ### Do you have any references?
   
   @sijie is my reference :-) 
   
   ### Would you like to fix this issue?
   
   Sorry - probably needs to be clarified by those who know more about the load balancer :-) 
   
   ### Note
   
   - [X] I have researched my question.


-- 
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.apache.org

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

[GitHub] [pulsar] momo-jun commented on issue #15904: [Doc] Load Balancer - Broker overload thresholds does not apply to some Shedding algorithms

Posted by GitBox <gi...@apache.org>.
momo-jun commented on issue #15904:
URL: https://github.com/apache/pulsar/issues/15904#issuecomment-1154811761

   @frankjkelly thanks for reporting this issue.
   
   As confirmed by @Demogorgon314, `loadBalancerBrokerOverloadedThresholdPercentage` applies to the `OverloadShedder` shedding algorithm only. A pull request will be submitted to update this.


-- 
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] momo-jun commented on issue #15904: [Doc] Load Balancer - Broker overload thresholds does not apply to some Shedding algorithms

Posted by GitBox <gi...@apache.org>.
momo-jun commented on issue #15904:
URL: https://github.com/apache/pulsar/issues/15904#issuecomment-1154945336

   > Great - thank you @momo-jun - will the PR be linked in here when submitted?
   
   Yes, you will be notified.


-- 
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] frankjkelly commented on issue #15904: [Doc] Load Balancer - Broker overload thresholds does not apply to some Shedding algorithms

Posted by GitBox <gi...@apache.org>.
frankjkelly commented on issue #15904:
URL: https://github.com/apache/pulsar/issues/15904#issuecomment-1160890925

   Thanks @Anonymitaet and @momo-jun 


-- 
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] frankjkelly commented on issue #15904: [Doc] Load Balancer - Broker overload thresholds does not apply to some Shedding algorithms

Posted by GitBox <gi...@apache.org>.
frankjkelly commented on issue #15904:
URL: https://github.com/apache/pulsar/issues/15904#issuecomment-1154929210

   Great - thank you @momo-jun - will the PR be linked in here when submitted?


-- 
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] Anonymitaet closed issue #15904: [Doc] Load Balancer - Broker overload thresholds does not apply to some Shedding algorithms

Posted by GitBox <gi...@apache.org>.
Anonymitaet closed issue #15904: [Doc] Load Balancer - Broker overload thresholds does not apply to some Shedding algorithms
URL: https://github.com/apache/pulsar/issues/15904


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