tustvold commented on pull request #1617: URL: https://github.com/apache/arrow-datafusion/pull/1617#issuecomment-1066672253
Running the benchmarks against a backported https://github.com/apache/arrow-rs/pull/1418 show the performance of this to be significantly worse than the current Sync reader (~2x). It is possible this is just a simple oversight with a simple fix, but I'm unlikely to have time to look into it this week. Async is far from free and so I'd expected the performance to be somewhat worse, but not that much worse... Hopefully I will have some time next week, otherwise I'll keep this pootling along on the backburner until it is ready for prime-time :smile: -- 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]
