AsyncDispatcher seems to have synchronization issues
torkleyy opened this issue · comments
This is the first time this occurred: https://travis-ci.org/slide-rs/shred/jobs/339916332#L606
A test calling mut_res panicked :(
cc @kvark I think the problem is that pulse is using Relaxed ordering, so the drop call of the resources Arc gets moved after the pulse.
Maybe we should consider removing AsyncDispatcher given that you can easily implement it yourself and the current version seems to be rather limiting.
cc @Object905