flightstats / hub

fault tolerant, highly available service for data storage and distribution
http://www.flightstats.com
MIT License
103 stars 35 forks source link

Add some concurrency-related safeties for StatsdReporter test #1154

Closed lkemmerer closed 5 years ago

lkemmerer commented 5 years ago

After Damon's changes, I was still getting consistent failures here. I threw some more concurrency-related stuff in and removed Thread.sleeps. It seems more stable now.

In the process of debugging, I ended up inlining the BeforeClass method, which likely decreases debuggability, but I found it difficult with all the CompletableFutures already. :-/