maropu commented on pull request #29643: URL: https://github.com/apache/spark/pull/29643#issuecomment-688062289
> The reason is that, WidenSetOperationTypes does the work by traversing the plan tree twice. Ah, good point. Yea, the current approach in this PR looks okay to me if all the existing tests pass. Btw, could we backport the previous PR and this PR into branch-3.0? The branch also has the issue described in SPARK-32638. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
