Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/22244
@cloud-fan, I like this solution better than adding a special case in the
v2 conversion to physical plan. This explains why the Python exec nodes weren't
already in the tree! I'd much rather commit this or something like it than go
with my work-around solution, which is fairly brittle.
For now, I'm going to assume that this is going to be the fix. Please let
me know if you want me to do further work to get my patch ready. We could use
it if this work ends up being more complex than we want to wait on for the
release.
@icexelloss, also feel free to take the test case from #22206, and thank
you for working on this!
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
