2010YOUY01 commented on PR #16889: URL: https://github.com/apache/datafusion/pull/16889#issuecomment-3123897115
> I didn't expect this PR to have such a big performance improvement. Like #16443, I still don't understand why there is a performance improvement. I’m aware of two key differences: - Fewer redundant steps for `indices <--> batches` - Always keeping the right batch in cache (the original implementation performs a left-chunk × right-row iteration) However I was just hoping to cleanup the codebase a bit, I also didn’t expect it to be an easy 2X. -- 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]
