jameslittle230 / stork

🔎 Impossibly fast web search, made for static sites.
https://stork-search.net
Apache License 2.0
2.73k stars 56 forks source link

Convert to BTreeMap right before serialization #266

Closed jameslittle230 closed 2 years ago

jameslittle230 commented 2 years ago

Fixes #261 Alternative to #265

codecov[bot] commented 2 years ago

Codecov Report

Merging #266 (91412a4) into master (eeaca67) will decrease coverage by 0.09%. The diff coverage is 25.00%.

@@            Coverage Diff             @@
##           master     #266      +/-   ##
==========================================
- Coverage   72.44%   72.34%   -0.10%     
==========================================
  Files          53       54       +1     
  Lines        2174     2177       +3     
  Branches      104      104              
==========================================
  Hits         1575     1575              
- Misses        598      601       +3     
  Partials        1        1              
Impacted Files Coverage Δ
stork-lib/src/index_v3/mod.rs 70.37% <ø> (ø)
stork-lib/src/index_v3/ordered_map_serialize.rs 0.00% <0.00%> (ø)
stork-lib/src/index_v3/build/mod.rs 98.57% <100.00%> (ø)

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 a7b8bdd...91412a4. Read the comment docs.

github-actions[bot] commented 2 years ago

Benchmarks

BenchmarkBaselineContenderComparison
build/federalist157.8962188.39121.19×
federalist.st1125.4561125.4561.0×
search/federalist/liberty1.57191.79791.14×
stork.js21.8821.881.0×
stork.wasm345.002344.9831.0×

Baseline: a7b8bdd69115b812dd9c44909139b146157f6038; Comparison: 91412a4992d0404586337b41b176c2cfe9c8ae3b

jameslittle230 commented 2 years ago

Closing; benchmarks for this were worse than benchmarks for #265