Panic (deadlock) on Parity shutdown when using Tendermint engine

[debris]

basically the same as #8088

[debris]

This case is actually slightly more complicated than #8088 . TransitionHandler from engines/transition.rs is generic over Engine<Machine>. The ideal solution would be one of the following:

a) internal refactor of out event loop
b) injecting event loop to an engine, so an engine never has an ownership of event loop (it shouldn't imo)
c) decoupling step logic from engine (I don't know if it's possible)

[5chdn]

#8664 :)

[Tbaut]

Closing because of #8664 :)

[debris]

I don't think it should be closed. The issue exists in current version and we don't have substrate PoC-1 implementation yet. And there is no guarantee that substrate PoC-1 implementation is not built as an alteration of tendermint engine and it does not have the same issue

[Tbaut]

I really think this issue is not worth investing time on it. Also #9980 totally removed Tendermint from Parity Ethereum.