pepijnve commented on PR #16196: URL: https://github.com/apache/datafusion/pull/16196#issuecomment-2940587625
Just FYI, I've completed (I think at least) the exercise of adapting the operators in https://github.com/pepijnve/datafusion/tree/cancel_safety. I wanted to finish that if only for the learning experience. Honestly, I don't think it's that dramatic a code change and intuitively at least I would expect overhead to be negligible. I would be interested to measure what the actual impact is on the benchmarks though and compare that to the variant here. @alamb is there any particular environment required to run that branch comparison script? -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
