Skip to content

Let fork GC call vecsim tiered index GC#3657

Merged
alonre24 merged 54 commits intomasterfrom
alon_gc_with_vecsim_tiered_index
Jul 13, 2023
Merged

Let fork GC call vecsim tiered index GC#3657
alonre24 merged 54 commits intomasterfrom
alon_gc_with_vecsim_tiered_index

Conversation

@alonre24
Copy link
Copy Markdown
Collaborator

Let fork GC call a new API in VecSim lib that allows calling the GC of all tiered indexes that are associated with some field in the index schema.

DvirDukhan
DvirDukhan previously approved these changes Jul 13, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 13, 2023

Codecov Report

Patch coverage: 95.45% and project coverage change: +0.02 🎉

Comparison is base (03ce9bb) 82.51% compared to head (a7ce965) 82.53%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3657      +/-   ##
==========================================
+ Coverage   82.51%   82.53%   +0.02%     
==========================================
  Files         192      192              
  Lines       32233    32255      +22     
==========================================
+ Hits        26597    26622      +25     
+ Misses       5636     5633       -3     
Impacted Files Coverage Δ
src/vector_index.c 85.28% <94.44%> (+0.47%) ⬆️
src/fork_gc.c 55.91% <100.00%> (+0.19%) ⬆️
src/gc.c 93.54% <100.00%> (+0.07%) ⬆️

... and 3 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.

@alonre24 alonre24 merged commit 88ea110 into master Jul 13, 2023
@alonre24 alonre24 deleted the alon_gc_with_vecsim_tiered_index branch July 13, 2023 21:48
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.

3 participants