alamb commented on PR #22914: URL: https://github.com/apache/datafusion/pull/22914#issuecomment-5271023014
> @alamb @avantgardnerio I've been thinking that I should revisit this and split it into smaller pieces that are easier to review. I think the idea fundamentally has merit; I'm happily accepting code reviews now, of course 😊, but I can refactor it first if that's easier to review. Thank you -- that would be helpful. However I don't want to lead you on -- I am not super familiar with the DataFusion join code and thus it will take me a while to review changes to it (and therefore given my limited time I will likely be a huge bottleneck / not reliable reviewer) If we want to make real progress on joins (which I do think is strategically quite important) maybe we can find a group of committers / reviewers who are interested in working together to push things along I tried to organize such a team before in an issue, but nothing came of it: - https://github.com/apache/datafusion/issues/15885 -- 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]
