beliefer commented on PR #43806:
URL: https://github.com/apache/spark/pull/43806#issuecomment-1812439348

   > Ideally that should have happened, but what I see is one stage containing 
subquery adaptive exec and incoming exchange contains subqueryexec. Also this 
is just 1 of the issues. The main pr will be the one which requires additional 
functions to be implemented by DataSourceV2impl. Pls refer to the ticket which 
depends on this pr. Mat be this itself can be fixed by ensuring incoming 
exchange contains right exec. Then tinkering with canonicalize may not be 
needed. But in that case the buildPlan should be canonicalized
   
   That is impossible. `SubqueryAdaptiveBroadcastExec` will be eliminate with 
`DynamicPruningExpression`.


-- 
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: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to