milenkovicm opened a new pull request, #1300:
URL: https://github.com/apache/datafusion-ballista/pull/1300
# Which issue does this PR close?
Closes #.
# Rationale for this change
At the moment `UnresolvedShuffleExec` does not support `with_new_children`
which is wrong, `with_new_children` should work if number of children is zero.
At the moment, ballista just splits the physical plan into stages but as it
is demonstrated in #1296 we would need to re-optimise some stages in order to
be able to fix #1296 (stage plans may not be correct once they are split)
# What changes are included in this PR?
- implement `UnresolvedShuffleExec::with_new_children`
- add check if number of children is 0 when `with_new_children` is called
for ShuffleWrite and Read
# Are there any user-facing changes?
No
--
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-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
