amahussein commented on pull request #2601: URL: https://github.com/apache/hadoop/pull/2601#issuecomment-756839310
@jbrennan333 The failing unit test is not related. I will take note and watch for it to file a jira if it shows up in the qbt-report. I have intuition that the equal-weighted nodes should be shuffled to maintain load balancing. In order to avoid the redundancy of shuffling twice, I changed the default `secondSorter` to null. I made quick evaluation of the performance using a single threaded. The new changes show less variance in code performance and slightly lower average. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
