Skip to content

Commit

Permalink
fix(driver): don't call wake_by_ref in Poll::Pending (#350)
Browse files Browse the repository at this point in the history
  • Loading branch information
sundy-li authored Feb 2, 2024
1 parent 22eb764 commit 2004b52
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
5 changes: 1 addition & 4 deletions driver/src/flight_sql.rs
Original file line number Diff line number Diff line change
Expand Up @@ -377,10 +377,7 @@ impl Stream for FlightSQLRows {
}
Poll::Ready(Some(Err(err))) => Poll::Ready(Some(Err(err.into()))),
Poll::Ready(None) => Poll::Ready(None),
Poll::Pending => {
cx.waker().wake_by_ref();
Poll::Pending
}
Poll::Pending => Poll::Pending,
}
}
}
5 changes: 1 addition & 4 deletions driver/src/rest_api.rs
Original file line number Diff line number Diff line change
Expand Up @@ -276,10 +276,7 @@ impl Stream for RestAPIRows {
self.next_page = None;
Poll::Ready(Some(Err(e)))
}
Poll::Pending => {
cx.waker().wake_by_ref();
Poll::Pending
}
Poll::Pending => Poll::Pending,
},
None => match self.next_uri {
Some(ref next_uri) => {
Expand Down

0 comments on commit 2004b52

Please sign in to comment.