You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Just checked the source code, activation and deactivation operations are relying on the CoinsContext Mutex.
e.g.,:
activation requested -> TendermintCoin initialized -> spawning tx/history / balance streaming futures -> locked(no parallel execution allowed here) coins context and added TendermintCoin in there -> success result
However, spawned futures can end up unexpectedly because there is no guarantee that they will be successfully executed/initialized before we add their coin into CoinsContext. This means that a spawned future could encounter errors or panics after we add the coin to the context and return a success result.
shamardy
changed the title
Return error if transaction history futures initialization fails
return an error if transaction history futures initialization fails
Oct 9, 2023
As explained by @onur-ozkan
ref. #1978 (comment), #1978 (comment), #1978 (comment)
The text was updated successfully, but these errors were encountered: