andygrove commented on PR #1537: URL: https://github.com/apache/datafusion-ballista/pull/1537#issuecomment-4366682061
> Thanks for the contribution @hcrosse! I'd like to help review and test this, but there have been substantial changes to shuffle so this PR likely needs rebasing and updating to work with sort-based shuffle, which is now the default. Actually, these changes are specific to the hash-based shuffle implementation. Could you fix the current conflict and I can help test with the standalone shuffle benchmark binary that was recently added. We can then consider a second PR to apply the same improvement to sort-based shuffle. WDYT? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
