ddanielr commented on PR #5998: URL: https://github.com/apache/accumulo/pull/5998#issuecomment-3665441252
> I'm not sure we should make this change in a patch release without making it configurable and defaulting to `off`. I think this thread contention issue - where you have multiple scans hitting the same tablet at the same time resulting in the same set of files in the same order - likely only happens in a few cases: > > ``` > - a table with very few tablets > - a case where you have a thundering herd of like queries starting at the same time > ``` Changed the code to move the shuffle to a different iterator class and handle the property lookup at the table level. Added back in the property and set it to false for a default value. Moving the functionality to a separate class was helpful as I could better track uses of MultiIterator to see where shuffling might be utilized. -- 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]
