devinjdangelo commented on PR #7562: URL: https://github.com/apache/arrow-datafusion/pull/7562#issuecomment-1722232402
Thanks @alamb for the review. I agree with the plan of action you lay out! I also merged in the changes from the alternative implementation PR given that it is about 10% faster and slightly lower memory usage in the release build benchmarks. I set the config default to false and expanded the doc comment to flag the missing features if you enable it. -- 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]
