pepijnve commented on PR #16196:
URL: https://github.com/apache/datafusion/pull/16196#issuecomment-2934052479

   > It's more complex than i expected, i need more time to investigate about 
the rule plan, because we add a new YieldStreamExec.
   > 
   > Several exec will have specified rule to manage the behaviour according 
the child of it. For example:
   > ...
   > We need more effort to bypass the YiledStreamExec, because it will affect 
other optimization.
   
   @ozankabak @zhuqi-lucas this does make me wonder again if making a support 
functions and adding this directly in the stream implementations isn't the 
simpler and easier to maintain solution after all. If it's ok with you guys I 
would like to experiment with a more intrusive solution to see what that would 
look like.


-- 
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

Reply via email to