Browse Source

Reduce `eventstream` timeout as it is linked to Python cleanup race condition that is cleaning up node first and then python arrow reference.

tags/0.3.8-rc
haixuanTao 1 year ago
parent
commit
45a7ee9437
1 changed files with 8 additions and 1 deletions
  1. +8
    -1
      apis/rust/node/src/event_stream/thread.rs

+ 8
- 1
apis/rust/node/src/event_stream/thread.rs View File

@@ -60,7 +60,14 @@ impl Drop for EventStreamThreadHandle {
if self.handle.is_empty() {
tracing::trace!("waiting for event stream thread");
}
match self.handle.recv_timeout(Duration::from_secs(20)) {

// TODO: The event stream duration has been shorten due to
// Python Reference Counting not working properly and deleting the node
// before deleting event creating a race condition.
//
// In the future, we hope to fix this issue so that
// the event stream can be properly waited for every time.
match self.handle.recv_timeout(Duration::from_secs(1)) {
Ok(Ok(())) => {
tracing::trace!("event stream thread finished");
}


Loading…
Cancel
Save