milleruntime commented on pull request #2117: URL: https://github.com/apache/accumulo/pull/2117#issuecomment-853329203
I successfully tested the recovery locally using Uno with TestIngest of 50000 rows and using two tservers. Ran all the ITs again and `WriteAheadLogEncryptedIT` was the only one failing. I refactored to pass server context in order to get the configuration and it now passes. There may be room for future performance improvements but I think the equivalent functionality of sorting recovery with RFile is ready to merge. This change should take care of #2076 as well but I want to do further testing with encryption since I don't think it works exactly how we want it to yet. I can do that as a follow on task. -- 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]
