crepererum commented on code in PR #4820:
URL: https://github.com/apache/arrow-datafusion/pull/4820#discussion_r1061417377
##########
datafusion/core/src/physical_plan/repartition.rs:
##########
@@ -555,20 +584,35 @@ impl Stream for RepartitionStream {
mut self: Pin<&mut Self>,
cx: &mut Context<'_>,
) -> Poll<Option<Self::Item>> {
- match self.input.poll_next_unpin(cx) {
Review Comment:
the change here isn't as large as it seems. I've just converted the implicit
loop via (tail) recursion into a proper one since I'm kinda afraid that this
may explode under certain circumstances. See one of the early commits in this
PR. The actual accounting change is in a separate commit.
--
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...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
