Manno15 edited a comment on pull request #2181: URL: https://github.com/apache/accumulo/pull/2181#issuecomment-878653866
I added a unit test in my latest commit. I ultimately decided to keep it on the simple side since `SortedLogRecoveryTest` is a bit more complex and recovery utilizes RLI so I feel it is adequately tested there. I do however test the features of RLI, such as validateFirstKey. I also test the error cases, such as trying to read a single file and trying to iterate through a directory that doesn't contain the finished marker. EDIT: Looking into the build error now. -- 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]
