geoffreyclaude commented on PR #19390: URL: https://github.com/apache/datafusion/pull/19390#issuecomment-4244317230
@adriangb Thanks for looking into this a bit more! If we agree most of these optimizations are worth pursuing (my biased Datadog opinion being that the Int8/Int16 and Utf8 optims are the most critical), we should first get the first two commits merged. Those set the foundations for the rest. Each subsequent optimization commit is then much easier to reason about independently and they can be easily split into dedicated PRs. The first commit is "just" AI generated benchmarks, I don't think we need to care too much about that one. The second one though is the most important... -- 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]
