alamb commented on PR #5322: URL: https://github.com/apache/arrow-datafusion/pull/5322#issuecomment-1436964442
> @alamb, if you can mention any collaborator who is interested in this side of things, maybe we can make some progress until you can make some time for this. I think @liukun4515 / @jackwener / @ygf11 / @mingmwang have all worked on joins recently, so they may enjoy seeing a symmetric hash join implementation @Dandandan has historically worked on the Join operators, so he may have some insight. In terms of being able to get the bandwidth to review this PR, the blog is a great first step (I look forward to reading it). Something that may also help would be to break this PR into several smaller PRs if possible (I haven't looked at the code yet so I don't know how easy / hard this would be to do). The rationale for more smaller PRs is that they require less contiguous time to review. For me, at least, finding 10 minutes to review a small PR is much easier to find 2 hours to review a larger one. Other reviewers may have a different opinion, of course, but that is mine -- 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]
