[GitHub] [hbase] ndimiduk commented on a change in pull request #2490: HBASE-24419 Normalizer merge plans should consider more than 2 regions when possible

2020-10-21 Thread GitBox
ndimiduk commented on a change in pull request #2490: URL: https://github.com/apache/hbase/pull/2490#discussion_r509692111 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/SimpleRegionNormalizer.java ## @@ -315,35 +316,60 @@ private boolean sk

[GitHub] [hbase] ndimiduk commented on a change in pull request #2490: HBASE-24419 Normalizer merge plans should consider more than 2 regions when possible

2020-10-09 Thread GitBox
ndimiduk commented on a change in pull request #2490: URL: https://github.com/apache/hbase/pull/2490#discussion_r502560699 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/SimpleRegionNormalizer.java ## @@ -315,35 +317,61 @@ private boolean sk

[GitHub] [hbase] ndimiduk commented on a change in pull request #2490: HBASE-24419 Normalizer merge plans should consider more than 2 regions when possible

2020-10-05 Thread GitBox
ndimiduk commented on a change in pull request #2490: URL: https://github.com/apache/hbase/pull/2490#discussion_r499937265 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/RegionNormalizerManager.java ## @@ -0,0 +1,153 @@ +/* + * Licensed to t

[GitHub] [hbase] ndimiduk commented on a change in pull request #2490: HBASE-24419 Normalizer merge plans should consider more than 2 regions when possible

2020-10-05 Thread GitBox
ndimiduk commented on a change in pull request #2490: URL: https://github.com/apache/hbase/pull/2490#discussion_r499920654 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/RegionNormalizerManager.java ## @@ -0,0 +1,153 @@ +/* + * Licensed to t

[GitHub] [hbase] ndimiduk commented on a change in pull request #2490: HBASE-24419 Normalizer merge plans should consider more than 2 regions when possible

2020-10-05 Thread GitBox
ndimiduk commented on a change in pull request #2490: URL: https://github.com/apache/hbase/pull/2490#discussion_r499913399 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/RegionNormalizerManager.java ## @@ -0,0 +1,153 @@ +/* + * Licensed to t

[GitHub] [hbase] ndimiduk commented on a change in pull request #2490: HBASE-24419 Normalizer merge plans should consider more than 2 regions when possible

2020-10-05 Thread GitBox
ndimiduk commented on a change in pull request #2490: URL: https://github.com/apache/hbase/pull/2490#discussion_r499913092 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/SplitTableRegionProcedure.java ## @@ -570,8 +569,10 @@ private void pre

[GitHub] [hbase] ndimiduk commented on a change in pull request #2490: HBASE-24419 Normalizer merge plans should consider more than 2 regions when possible

2020-10-05 Thread GitBox
ndimiduk commented on a change in pull request #2490: URL: https://github.com/apache/hbase/pull/2490#discussion_r499912755 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java ## @@ -1953,20 +1952,27 @@ public SetNormalizerRunningRespo

[GitHub] [hbase] ndimiduk commented on a change in pull request #2490: HBASE-24419 Normalizer merge plans should consider more than 2 regions when possible

2020-10-05 Thread GitBox
ndimiduk commented on a change in pull request #2490: URL: https://github.com/apache/hbase/pull/2490#discussion_r499912573 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java ## @@ -3003,18 +2933,20 @@ public double getAverageLoad() { retu