tgravescs commented on pull request #29224: URL: https://github.com/apache/spark/pull/29224#issuecomment-663616748
thanks for looking @dongjoon-hyun, I went back and forth on the improvement vs bug. I can see it both ways, I decided to file it as a bug because without it we can't properly override AQE as intended with the original change. It would be nice to pull these back to branch-3.0 as well since many people are starting to use AQE there and was thinking the change was pretty isolated to that code path. ---------------------------------------------------------------- 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. 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