comphead commented on PR #17482: URL: https://github.com/apache/datafusion/pull/17482#issuecomment-3282318389
Thanks @jonathanc-n and @2010YOUY01 #17467 definitely would be nice to have as PWMJ can start as optional experimental join, which would be separately documented, showing benefits and limitations for the end user. Actually the same happened for SMJ being experimental feature for quite some time. Another great point to identify bottlenecks in performance is to absorb some knowledge from https://github.com/apache/datafusion/issues/17488 and keep the join more stable. As optional feature it is pretty safe to go, again referring to SMJ there was a separate ticket which post launch checks to make sure it is safe to use like https://github.com/apache/datafusion/issues/9846 Let me know your thoughts? -- 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]
