ctubbsii commented on issue #1791: URL: https://github.com/apache/accumulo/issues/1791#issuecomment-764783383
> In the for loop, the break condition is the same as the do-while loop meaning once the for loop is finished the do-while will only execute once(essentially rendering it useless). It does call `waitForBalance` before entering this second loop, so even though the loop condition is the same, I think the expectation is that `waitForBalance` call could make the condition not true temporarily, hence the second loop. I'm not sure if this 5 seconds is enough time to guarantee that balance has actually started. > After this happens, 2 servers are killed which sometimes takes a bit of time. This may be the window of time in which the tablets migrate under the radar. Hopefully not. Once balancing has settled, there should be little reason for them to migrate. > I have tried increasing this amount of time and haven't gotten the error since, however this error doesn't happen too often as is so its hard to tell if this is definitely the cause. You could try shortening the time to see if the error appears more often afterwards. That might help indicate that increasing the time is definitely going to improve the stability of the test and is the correct path forward. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
