logv / sybil

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

reduce use of opts #70

Closed tmc closed 6 years ago

tmc commented 6 years ago

as part of global reference reduction this removes some of the use of OPTS

codecov-io commented 6 years ago

Codecov Report

Merging #70 into master will decrease coverage by 0.02%. The diff coverage is 51.35%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
- Coverage   52.46%   52.44%   -0.03%     
==========================================
  Files          36       37       +1     
  Lines        3709     3703       -6     
==========================================
- Hits         1946     1942       -4     
- Misses       1569     1570       +1     
+ Partials      194      191       -3
Impacted Files Coverage Δ
src/sybil/config.go 75.43% <ø> (-2.34%) :arrow_down:
src/sybil/printer.go 0% <0%> (ø) :arrow_up:
src/sybil/table_trim.go 0% <0%> (ø) :arrow_up:
src/sybil/table_lock.go 53.01% <0%> (ø) :arrow_up:
src/sybil/replacements.go 0% <0%> (ø)
src/sybil/table_query.go 64.15% <100%> (+4.42%) :arrow_up:
src/sybil/table_block_io.go 56.61% <66.66%> (ø) :arrow_up:
src/sybil/column_store_io.go 79.61% <92.3%> (+0.28%) :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 9a5f93c...12ad67f. Read the comment docs.