uber / aresdb

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

ShardOwner interface #220

Closed jshencode closed 5 years ago

jshencode commented 5 years ago

use ShardOwner interface to remove hard coded shards

during query time, if query specified shards, use query specified shards for query if query does not specify shards, use owned shards

codecov-io commented 5 years ago

Codecov Report

Merging #220 into master will decrease coverage by 0.1%. The diff coverage is 42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #220      +/-   ##
==========================================
- Coverage   73.77%   73.66%   -0.11%     
==========================================
  Files         152      153       +1     
  Lines       20591    20615      +24     
==========================================
- Hits        15190    15187       -3     
- Misses       4249     4276      +27     
  Partials     1152     1152
Impacted Files Coverage Δ
query/common/aql.go 0% <ø> (ø) :arrow_up:
memstore/memstore.go 68.45% <ø> (ø) :arrow_up:
metastore/disk_metastore.go 75.83% <ø> (-0.07%) :arrow_down:
datanode/datanode.go 0% <0%> (ø) :arrow_up:
broker/executor.go 0% <0%> (ø) :arrow_up:
cluster/topology/shard_owner.go 0% <0%> (ø)
memstore/recovery.go 75.26% <100%> (+0.52%) :arrow_up:
api/query_handler.go 72.98% <100%> (+0.12%) :arrow_up:
memstore/batchstats.go 86.2% <100%> (+2.87%) :arrow_up:
query/aql_compiler.go 75.9% <81.81%> (-0.07%) :arrow_down:
... and 3 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 3508f97...fa8ecbb. Read the comment docs.