agavra commented on PR #9773: URL: https://github.com/apache/pinot/pull/9773#issuecomment-1312922599
@walterddr i had to force push because of rebasing and some refs I messed up locally, but the changes since you last reviewed: - I figured out the timeout issues, it was because i was using `LinkedList` instead of `ArrayList` in the `HashExchange` code. Turns out it's dramatically (think 100x slower) to do that when you have lots of rows - I changed the pattern to iterator as discussed - I made a few changes to account for the new `isLeafStageSender` flag that was introduced in your other PR (merge conflict) -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org