Re: [PR] fix: repair AQE plan rewrites, remove a redundant shuffle, and enable adaptive query planning by default [datafusion-ballista]

["milenkovicm (via GitHub)" <gi...@apache.org>]

milenkovicm commented on code in PR #2315:
URL: https://github.com/apache/datafusion-ballista/pull/2315#discussion_r3791698369


##########
ballista/client/tests/context_checks.rs:
##########
@@ -1074,6 +1074,13 @@ mod supported {
         #[case]
         ctx: SessionContext,
     ) {
+        // Golden plan text is the static planner's stage layout.

Review Comment:
   note for follow up: we need to check `EXPLAN` format with AQE



-- 
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-unsubscribe@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org