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

check controller enabled #261

Closed lxning closed 5 years ago

lxning commented 5 years ago

Ignore jobConfig is empty if controller is enabled during initialization.

codecov-io commented 5 years ago

Codecov Report

Merging #261 into master will decrease coverage by 0.02%. The diff coverage is 80%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #261      +/-   ##
==========================================
- Coverage   68.85%   68.82%   -0.03%     
==========================================
  Files         163      163              
  Lines       22610    22607       -3     
==========================================
- Hits        15567    15560       -7     
- Misses       5823     5827       +4     
  Partials     1220     1220
Impacted Files Coverage Δ
subscriber/common/job/controller.go 41.89% <0%> (ø) :arrow_up:
subscriber/config/service_config.go 82.22% <100%> (-1.12%) :arrow_down:
datanode/peer_source.go 58.11% <0%> (-2.57%) :arrow_down:
cluster/topology/dynamic.go 70.4% <0%> (-2.4%) :arrow_down:
controller/mutators/etcd/schema_mutator.go 59.17% <0%> (-1.78%) :arrow_down:
subscriber/common/job/driver.go 71.98% <0%> (+0.7%) :arrow_up:
broker/query_plan_non_agg.go 76.03% <0%> (+2.47%) :arrow_up:

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 f46f9db...0fb677f. Read the comment docs.