Skip to content

Commit

Permalink
post-merge fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
korowa committed Dec 16, 2023
1 parent f2b72f2 commit b164328
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 8 deletions.
5 changes: 1 addition & 4 deletions datafusion/physical-plan/src/joins/hash_join.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1159,10 +1159,7 @@ impl HashJoinStream {
fn poll_next_impl(
&mut self,
cx: &mut std::task::Context<'_>,
) -> Poll<Option<Result<RecordBatch>>>
where
Self: Send,
{
) -> Poll<Option<Result<RecordBatch>>> {
loop {
return match self.state {
HashJoinStreamState::WaitBuildSide => {
Expand Down
8 changes: 4 additions & 4 deletions datafusion/physical-plan/src/joins/stream_join_utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -759,7 +759,7 @@ pub trait EagerJoinStream {
match self.right_stream().next().await {
Some(Ok(batch)) => {
if batch.num_rows() == 0 {
return Ok(StreamJoinStateResult::Continue);
return Ok(StatefulStreamResult::Continue);
}

self.set_state(EagerJoinStreamState::PullLeft);
Expand Down Expand Up @@ -788,7 +788,7 @@ pub trait EagerJoinStream {
match self.left_stream().next().await {
Some(Ok(batch)) => {
if batch.num_rows() == 0 {
return Ok(StreamJoinStateResult::Continue);
return Ok(StatefulStreamResult::Continue);
}
self.set_state(EagerJoinStreamState::PullRight);
self.process_batch_from_left(batch)
Expand Down Expand Up @@ -817,7 +817,7 @@ pub trait EagerJoinStream {
match self.left_stream().next().await {
Some(Ok(batch)) => {
if batch.num_rows() == 0 {
return Ok(StreamJoinStateResult::Continue);
return Ok(StatefulStreamResult::Continue);
}
self.process_batch_after_right_end(batch)
}
Expand Down Expand Up @@ -847,7 +847,7 @@ pub trait EagerJoinStream {
match self.right_stream().next().await {
Some(Ok(batch)) => {
if batch.num_rows() == 0 {
return Ok(StreamJoinStateResult::Continue);
return Ok(StatefulStreamResult::Continue);
}
self.process_batch_after_left_end(batch)
}
Expand Down

0 comments on commit b164328

Please sign in to comment.