I have some questions;

1. Does this mean we'll share the harvest thread between job A and B? (ray.init() (job A) => ray.shutdown() => ray.init() (job B)).
2. Also I am concerned that not terminating the harvest thread will cause weird issues like segfault on shutdown hook. If the harvest thread touches any other parts that are shut down before the process terminates, it can cause weird issues.
3. Also, why can't we restart the thread? Isn't it just we kill all threads => we restart? What prevents us?

Also @ashione Please take a look!

I have some questions;

1. Does this mean we'll share the harvest thread between job A and B? (ray.init() (job A) => ray.shutdown() => ray.init() (job B)).
2. Also I am concerned that not terminating the harvest thread will cause weird issues like segfault on shutdown hook. If the harvest thread touches any other parts that are shut down before the process terminates, it can cause weird issues.
3. Also, why can't we restart the thread? Isn't it just we kill all threads => we restart? What prevents us?

Also @ashione Please take a look!

In my experience, we'd better to fix the above issues rather than just skip shutdown or make harvest thread as a detached thread.
@simon-mo Could you give us some clues about why these will happend.

Please hold on. I'll try to figure out why opencensus thread crashed before next weekend.

@ashione any update?

@ashione any update?

I have encountered the same problem and am looking for a solution. We'd better restart the thread.

@ashione any update?

@rkooo567 @simon-mo I add some unit tests in metric exporter client test, but no such failures? In that case, ray stats had been setup and shutdown twice.

@ashione any update?

@rkooo567 @simon-mo I add some unit tests in metric exporter client test, but no such failures? In that case, ray stats had been setup and shutdown twice.

#13708

ping @simon-mo

ping @simon-mo

We've tried to close DeltaProducer in #13708, but found some other problems. It may introduce new problems if we modify more codes in opencensus. So we suggest do not support reopen in driver.