Skip to content

Commit 0ce85fe

Browse files
authored
HBASE-24407 Correct the comment of clusterRegionLocationMocks in TestStochasticLoadBalancer (#1750)
1 parent e1d7dc8 commit 0ce85fe

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestStochasticLoadBalancer.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ public class TestStochasticLoadBalancer extends BalancerTestBase {
8383
new int[]{2, 0, 0, 100}, // region 0 is hosted and entirely local on server 2
8484
new int[]{0, 100, 0, 0}, // region 1 is hosted and entirely on server 0
8585
new int[]{0, 100, 0, 0}, // region 2 is hosted and entirely on server 0
86-
new int[]{1, 0, 100, 0}, // region 1 is hosted and entirely on server 1
86+
new int[]{1, 0, 100, 0}, // region 3 is hosted and entirely on server 1
8787
},
8888

8989
// Test 2: each region is 0% local on the server that hosts it
@@ -92,7 +92,7 @@ public class TestStochasticLoadBalancer extends BalancerTestBase {
9292
new int[]{0, 0, 0, 100}, // region 0 is hosted and entirely local on server 2
9393
new int[]{1, 100, 0, 0}, // region 1 is hosted and entirely on server 0
9494
new int[]{1, 100, 0, 0}, // region 2 is hosted and entirely on server 0
95-
new int[]{2, 0, 100, 0}, // region 1 is hosted and entirely on server 1
95+
new int[]{2, 0, 100, 0}, // region 3 is hosted and entirely on server 1
9696
},
9797

9898
// Test 3: each region is 25% local on the server that hosts it (and 50% locality is possible)
@@ -101,7 +101,7 @@ public class TestStochasticLoadBalancer extends BalancerTestBase {
101101
new int[]{0, 25, 0, 50}, // region 0 is hosted and entirely local on server 2
102102
new int[]{1, 50, 25, 0}, // region 1 is hosted and entirely on server 0
103103
new int[]{1, 50, 25, 0}, // region 2 is hosted and entirely on server 0
104-
new int[]{2, 0, 50, 25}, // region 1 is hosted and entirely on server 1
104+
new int[]{2, 0, 50, 25}, // region 3 is hosted and entirely on server 1
105105
},
106106

107107
// Test 4: each region is 25% local on the server that hosts it (and 100% locality is possible)
@@ -110,7 +110,7 @@ public class TestStochasticLoadBalancer extends BalancerTestBase {
110110
new int[]{0, 25, 0, 100}, // region 0 is hosted and entirely local on server 2
111111
new int[]{1, 100, 25, 0}, // region 1 is hosted and entirely on server 0
112112
new int[]{1, 100, 25, 0}, // region 2 is hosted and entirely on server 0
113-
new int[]{2, 0, 100, 25}, // region 1 is hosted and entirely on server 1
113+
new int[]{2, 0, 100, 25}, // region 3 is hosted and entirely on server 1
114114
},
115115

116116
// Test 5: each region is 75% local on the server that hosts it (and 75% locality is possible everywhere)
@@ -119,7 +119,7 @@ public class TestStochasticLoadBalancer extends BalancerTestBase {
119119
new int[]{0, 75, 75, 75}, // region 0 is hosted and entirely local on server 2
120120
new int[]{1, 75, 75, 75}, // region 1 is hosted and entirely on server 0
121121
new int[]{1, 75, 75, 75}, // region 2 is hosted and entirely on server 0
122-
new int[]{2, 75, 75, 75}, // region 1 is hosted and entirely on server 1
122+
new int[]{2, 75, 75, 75}, // region 3 is hosted and entirely on server 1
123123
},
124124
};
125125

0 commit comments

Comments
 (0)