search

logv / sybil

columnar storage + NoSQL OLAP engine | https://logv.org
https://logv.org
Other
305 stars 24 forks source link

Parallelize tests, remove data races. #57

Closed tmc closed 6 years ago

tmc commented 6 years ago

This changes the tests to run in parallel and goes through and refactors all the code to remove detected races

Further work here would be to for parallelism within a test case (likely using t.Run()) which will make #52 more relevant.

This is groundwork to support multiple queries within the same address space, namely for #42

tmc commented 6 years ago

So this got a little big and unwieldy. I opened https://github.com/logv/sybil/pull/61 to be a small part of this work and think it will be easier to review in smaller batches.