search

uber / aresdb

A GPU-powered real-time analytics storage and query engine.
https://eng.uber.com/aresdb/
Apache License 2.0
3.01k stars 234 forks source link

Fix dependency #270

Closed jshencode closed 5 years ago

jshencode commented 5 years ago

vectors has dependency on cgoutils and cannot be placed in common

move vector into vectors package move testFactoryBase into tests package

codecov-io commented 5 years ago

Codecov Report

Merging #270 into master will decrease coverage by 0.11%. The diff coverage is 67.3%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #270      +/-   ##
==========================================
- Coverage   72.06%   71.95%   -0.12%     
==========================================
  Files         167      166       -1     
  Lines       22899    22756     -143     
==========================================
- Hits        16503    16373     -130     
+ Misses       5131     5121      -10     
+ Partials     1265     1262       -3
Impacted Files Coverage Δ
memstore/list/vector_party.go 64.4% <ø> (ø) :arrow_up:
memstore/vectors/vector.go 0% <0%> (ø)
memstore/live_store.go 84.11% <100%> (ø) :arrow_up:
memstore/archive_store.go 79.66% <100%> (ø) :arrow_up:
memstore/merge.go 96.65% <100%> (ø) :arrow_up:
memstore/vector_party.go 71.72% <100%> (ø) :arrow_up:
memstore/test_factory.go 70.7% <100%> (ø) :arrow_up:
memstore/list/test_factory.go 57.5% <100%> (ø) :arrow_up:
memstore/backfill.go 74.86% <100%> (ø) :arrow_up:
memstore/list/live_vector_party.go 60.09% <100%> (ø) :arrow_up:
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1ac619a...7585a96. Read the comment docs.