mhansonp commented on a change in pull request #7124:
URL: https://github.com/apache/geode/pull/7124#discussion_r782498356



##########
File path: 
geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/rebalance/PartitionedRegionLoadModelJUnitTest.java
##########
@@ -1417,9 +1467,16 @@ public void 
testTwoZoneRedundancyFourFullCopiesEnforceUniqueZones() throws Unkno
     assertThat(bucketOperator.removes).isEqualTo(expectedRemoves);
   }
 
-
+  /**
+   * Test that when you have five zones with one server each, with enforce 
unique zones, a
+   * redundancy
+   * level of 1, there is an overredundancy, and two servers have no data, 
that the overredundancy
+   * gets removed and buckets get moved to rebalance the data load
+   *
+   */
   @Test
-  public void testFiveZoneRedundancyFourFullCopiesEnforceUniqueZones() throws 
UnknownHostException {
+  public void fiveZonesWithRedundancyAndFourFullCopiesAndEnforceUniqueZones()

Review comment:
       Yup. Good catch.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to