ddanielr commented on PR #5998: URL: https://github.com/apache/accumulo/pull/5998#issuecomment-3618793837
Worked through this with @keith-turner and found that the shuffling could be done lower in the MultiIterator before things are added to the priorityQueue. Realized we had to do it in two places because of deepCopy. I added a test for deepCopy since that didn't exist. I'm not sure if we need to gate this change behind a property or not so I've removed that code for now. The MultiIterator is a system iterator so it's not part of the public API -- 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]
