[GitHub] merlimat commented on a change in pull request #1488: Load manager should offload multiple bundles when overloaded

[GitBox <gi...@apache.org>]

merlimat commented on a change in pull request #1488: Load manager should offload multiple bundles when overloaded
URL: https://github.com/apache/incubator-pulsar/pull/1488#discussion_r178684779
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java
 ##########
 @@ -365,7 +365,7 @@
     private boolean loadBalancerSheddingEnabled = true;
     // Load shedding interval. Broker periodically checks whether some traffic should be offload from some over-loaded
     // broker to other under-loaded brokers
-    private int loadBalancerSheddingIntervalMinutes = 5;
+    private int loadBalancerSheddingIntervalMinutes = 1;
 
 Review comment:
   I think it is safe to change this value because that is only the frequency for checking for overloaded brokers. 5 minutes can be a very long time for that. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services