digitalocean / firebolt

Golang framework for streaming ETL, observability data pipeline, and event processing apps
Other
702 stars 50 forks source link

Remove messaging panic, fix flaky integration test #17

Closed jnadler closed 4 years ago

jnadler commented 4 years ago

The panic() here was a bad idea. In fact, using singleton at all was a bad idea. For now, remove the panic to make it possible to write unit tests.

The ES integration test was failing with an EOF error intermittently. Add a small delay so that it succeeds consistently.