milleruntime commented on issue #2979: URL: https://github.com/apache/accumulo/issues/2979#issuecomment-1261281299
Yeah that looks like the problem. I made the `RecoveryLogsIterator` use the RFile API back when I changed recovery to use RFiles but that was prior to the crypto changes. We may have to make recovery use the internal File reader classes so we can pass in the CryptoService. I made the `LogSorter` create a CryptoService for writing the sorted log files but missed the code path when the tserver reads the sorted logs. We will have to do similar to what I did here: https://github.com/apache/accumulo/blob/42a6a9f4c54a60e73ec5b20841f883353c95d2e8/server/tserver/src/main/java/org/apache/accumulo/tserver/log/LogSorter.java#L226-L227 -- 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]
