ozankabak commented on PR #16196: URL: https://github.com/apache/datafusion/pull/16196#issuecomment-2934263065
@zhuqi-lucas, you may be having trouble with current rules because `YieldStreamExec` doesn't implement certain `ExecutionPlan` APIs. I would be surprised if the rules are that brittle and break for an operator that basically an *identity operator*. @pepijnve, an intrusive solution will be a hard sell for me. There are simply too many cases, each with their own context and somewhat specific details. I think @zhuqi-lucas is very close to achieving a good solution. However, feel free to experiment with your ideas if you think we can somehow cross-pollinate ideas and move towards an even better solution. -- 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
