MBF will block the main thread for an arbitrary time when canceling an execution

[corot]

Because we join its thread after cancelling an execution. This is dangerous because we relay on plugins cancel implementation to finish properly. And meanwhile we won't have callbacks, so for example a controller slowing down on cancel won't get an updated velocity from MBF.

UPDATE After #345, this problem won't happen if the old and new goals have the same controller. In that case, we will preempt the previous goal instead of canceling it.

[Nisarg236]

Hey @corot @siferati @renan028 , I was working on this issue and this implementation works: #359 but I wanted to know if you thought about something similar and there is any side effects of doing so.