cshannon opened a new pull request, #4203: URL: https://github.com/apache/accumulo/pull/4203
There fixes a test race condition in the testVerifyPassword test that could occasionally cause the test to fail if it ran too quickly. Token generation uses System.currentTimeMillis() so the test previously would fail when trying to generate a new token if the timestamp didn't update as the same token would be returned. This commit adds a short sleep of 1 ms to guarantee that the test won't fail. Before this fix when running that test in a loop in my IDE it would typically fail within 3-5 iterations. With this change I was able to run 10k iterations in a loop without failure. -- 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]
