jmark99 opened a new pull request, #3021: URL: https://github.com/apache/accumulo/pull/3021
This ticket closes #3017 (SortedLogRecoveryTest failures). That issue described a situation where a large number of tests within SortedLogRecoverTest.java were failing within certain environments. It failed consistently for me as well as on the Apache Jenkins server. The problem was tracked down to a issue where the 'getFiles' method within RecoveryLogsIterator was returning a List of logFiles that were presumed to be in lexicographical order by file name. But on my particular instance (and presumably the Jenkins server), the files were not returned in that order. Therefore when 'validateFirstKey' retrieved the first file from the provided list, it was not actually checking the first key, but instead, whatever key was in the first file provided by the list. In cases where the order was not as assumed, the check failed and an exception was thrown. The 'getFiles' method has been updated to return a SortedSet of files, thereby supplying 'validateFirstKey' with the logFiles in expected order. This change allowed all tests to pass without any additional code modification. -- 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]
