dolittle / Bifrost

This is the stable release of Dolittle till its out of alpha->beta stages
Other
110 stars 32 forks source link

Missing .Wait() on async calls in Table Storage EventStore #838

Closed svethors closed 7 years ago

svethors commented 7 years ago

TableStorage EventStore class constructor and method Commit has calls to async methods that are not awaited.

einari commented 7 years ago

One of the missing waits is by design; on Commit. No reason to wait for it to complete. But we might want to change this into something else - deal with async errors instead.