logv / sybil

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

add sybild grpc service #96

Closed okayzed closed 5 years ago

okayzed commented 5 years ago

originally from #42

okayzed commented 5 years ago

I'm not really understanding what is going on with the merge here. I resolved the conflicts locally, then I hit another set of merge conflicts. It seems like there are 3 or more conflicting commits when I rebase off master.

I spent part of yesterday re-validating the code in this commit (looks reasonable) and testing it on the snorkel grpc branch. I'm ready to merge it in as soon as we can make it past the conflicts

okayzed commented 5 years ago

landed on master in https://github.com/logv/sybil/commit/364be19d8ddae6ecf25f07b629d93d42bb55a29b