andygrove commented on issue #2859: URL: https://github.com/apache/datafusion-comet/issues/2859#issuecomment-3628554751
Nevermind, the logic is confusing but not incorrect. The reason is that `op.children.forall(_.isInstanceOf[CometNativeExec])` will always return true because query stages are leaf nodes and have no children. The code is confusing though, so I will create a PR to address this. -- 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]
