You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/10/14 12:51:24 UTC

[GitHub] [accumulo] dlmarion commented on pull request #3021: Change List to SortedSet in RecoveryLogsIterator

dlmarion commented on PR #3021:
URL: https://github.com/apache/accumulo/pull/3021#issuecomment-1278967672

   The other thing to note here is that this worked previously because RecoveryLogsIterator used the client side RFile scanner which took all the files from `Filesystem.listFiles` and then looked at the first key (internally it must have done a merge-sort read). I modified RecoveryLogsIterator in [this](https://github.com/apache/accumulo/commit/b14df03bd7eee697fa9002832f03c2aaa7442695#diff-288f570b1f810a8673289600ce7e47e6ea5727a612de5f0e7acb84a5f319d957R166) commit to use the internal RFile reader because it supports the CryptoService. It only takes one file, and I gave it the first one thinking they were sorted.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org