andygrove commented on PR #4538: URL: https://github.com/apache/datafusion-comet/pull/4538#issuecomment-4626658980
> Thanks for the reviews @comphead and @mbutrovich! Once this merges I will have small follow on to improve the design around the `allowIncompatCodegenDispatch` flag, which I find a little confusing Actually I need to fix this in the current PR. This PR inadvertently enabled codegen for more expressions that it appears. Moving to draft to avoid accidental merge. -- 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]
