apurtell commented on code in PR #8264:
URL: https://github.com/apache/hbase/pull/8264#discussion_r3290985978
##########
hbase-server/src/test/java/org/apache/hadoop/hbase/master/assignment/AssignmentTestingUtil.java:
##########
@@ -49,12 +49,17 @@ private AssignmentTestingUtil() {
}
public static void waitForRegionToBeInTransition(final HBaseTestingUtil util,
- final RegionInfo hri) throws Exception {
- while (!getMaster(util).getAssignmentManager().isRegionInTransition(hri)) {
+ final RegionInfo hri) {
+ while (!isRegionInTransition(hri, getMaster(util).getAssignmentManager()))
{
Threads.sleep(10);
Review Comment:
This is a good suggestion, if unrelated to the change, but might cause
issues with other tests once you start pulling on the thread. I would ignore
this one.
--
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]
