go-ego / riot

Go Open Source, Distributed, Simple and efficient Search Engine; Warning: This is V1 and beta version, because of big memory consume, and the V2 will be rewrite all code.
Apache License 2.0
6.11k stars 473 forks source link

Simplify LogicExpr api and update DocId type to string #72

Closed vcaesar closed 5 years ago

vcaesar commented 5 years ago
CLAassistant commented 5 years ago

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

codecov[bot] commented 5 years ago

Codecov Report

Merging #72 into master will increase coverage by 0.49%. The diff coverage is 97.32%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #72      +/-   ##
==========================================
+ Coverage   91.13%   91.63%   +0.49%     
==========================================
  Files           9       10       +1     
  Lines         857      932      +75     
==========================================
+ Hits          781      854      +73     
- Misses         53       54       +1     
- Partials       23       24       +1
Impacted Files Coverage Δ
ranker_worker.go 100% <ø> (ø) :arrow_up:
indexer_worker.go 100% <100%> (ø) :arrow_up:
segment.go 87.04% <100%> (ø) :arrow_up:
riot.go 95.06% <100%> (-0.29%) :arrow_down:
store_worker.go 88.88% <100%> (-0.77%) :arrow_down:
engine.go 90.1% <90%> (-0.06%) :arrow_down:
test_utils.go 97.61% <97.61%> (ø)

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 8e3e7f7...f5a0dc5. Read the comment docs.