NoahKusaba commented on PR #2438: URL: https://github.com/apache/datafusion-ballista/pull/2438#issuecomment-5612753240
The sensible thing would have been to split this up into many smaller PR's, but I don't want to spend much time on this, as I will be refocusing efforts on the iceberg-ballista integration, as the iceberg-datafusion crate has migrated to it's own repo. This is clearly important work that should get done, but I'm hoping the other more experienced maintainers can focus their efforts on feature integrations to close the gap with Spark's performance on benchmarks. Let me know how you would like me to proceed, or if we can just merge this as-is. I have read every line, and it makes sense to me. -- 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]
