incubator-pegasus icon indicating copy to clipboard operation
incubator-pegasus copied to clipboard

feat enhance : Ignore specified unhealthy tables during load balancing.

Open ruojieranyishen opened this issue 2 years ago • 2 comments

https://github.com/apache/incubator-pegasus/issues/1570

There are three modifications to implement the issue:

  1. Modifiy load balancing start condition.
-    if (healthy_partitions != total_partitions) {
+    if (total_partitions - healthy_partitions != ignore_app_unhealthy_partitions) {
  1. Refactor the existing app blacklist, and only transfer the required apps to the load balancer.
  2. Add unit tests. app_blacklist_with_balancer_test

ruojieranyishen avatar Jul 27 '23 08:07 ruojieranyishen

Hi @ruojieranyishen, thanks for the contribution!

Do you mind to separate the refactor (the point 3 you mentioned above) to another independent pull request? It would be nice the keep the pull requests to be more independent. Thanks!

acelyc111 avatar Oct 11 '23 16:10 acelyc111

@acelyc111 Thanks, I will take your advice and get it done as soon as possible.

ruojieranyishen avatar Oct 12 '23 01:10 ruojieranyishen