Skip to content

VecSim performance improvements#3701

Merged
alonre24 merged 12 commits intomasterfrom
vecsim_perf_improvements
Jul 13, 2023
Merged

VecSim performance improvements#3701
alonre24 merged 12 commits intomasterfrom
vecsim_perf_improvements

Conversation

@alonre24
Copy link
Copy Markdown
Collaborator

@alonre24 alonre24 commented Jul 9, 2023

Update vecsim version to the new one that contains performance improvement in HNSW (using vector blocks). This also contains changes due to a change in params struct (turned union field from anonymous to non-anonymous).

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 9, 2023

Codecov Report

Patch coverage: 91.66% and project coverage change: -0.01 ⚠️

Comparison is base (a0aed00) 82.50% compared to head (ae88e4f) 82.50%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3701      +/-   ##
==========================================
- Coverage   82.50%   82.50%   -0.01%     
==========================================
  Files         192      192              
  Lines       32233    32233              
==========================================
- Hits        26595    26593       -2     
- Misses       5638     5640       +2     
Impacted Files Coverage Δ
src/vector_index.c 84.81% <88.88%> (ø)
src/spec.c 89.02% <94.00%> (+0.12%) ⬆️
src/fields_global_stats.c 89.28% <100.00%> (ø)
src/json.c 89.71% <100.00%> (ø)

... and 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@DvirDukhan DvirDukhan marked this pull request as ready for review July 12, 2023 20:57
@alonre24 alonre24 merged commit 03ce9bb into master Jul 13, 2023
@alonre24 alonre24 deleted the vecsim_perf_improvements branch July 13, 2023 07:19
alonre24 added a commit that referenced this pull request Jul 16, 2023
alonre24 added a commit that referenced this pull request Jul 16, 2023
* VecSim performance improvements (#3701)

* Let fork GC call vecsim tiered index GC (#3657)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants