EdColeman opened a new pull request, #3616: URL: https://github.com/apache/accumulo/pull/3616
Adjusts two failing CI MemoryStarvedScanIT tests. Fixes #3598 The test started failing when Wait.waitFor() was modified to throw an exception. The tests previously were ignoring the condition was not successful. - testScanPauses - increases the slow iterator pause time to allow scanner time to detect / recover from low memory. - testBatchScanPauses increases number of rows so more than one batch is returned. Reduces slow iterator time, pause needs to be long enough to return one batch, but longer times just takes longer to gather the batch. Note - on one run I received a OOM / heap error, so there may be additional adjustments required, but I have not been able to reproduce. Forcing low memory and then recovery are difficult to reliably simulate, so this test may remain flaky, even with these changes. -- 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]
