Skip to content

[8.2] Update vectors memory in total index info memory [MOD-10841]#6641

Merged
meiravgri merged 1 commit into8.2from
backport-6613-to-8.2
Aug 15, 2025
Merged

[8.2] Update vectors memory in total index info memory [MOD-10841]#6641
meiravgri merged 1 commit into8.2from
backport-6613-to-8.2

Conversation

@alonre24
Copy link
Collaborator

@alonre24 alonre24 commented Aug 13, 2025

Describe the changes in the pull request

Backport of #6613 to 8.2.

Mark if applicable

  • This PR introduces API changes
  • This PR introduces serialization changes

* account for vector index in total memory info

* Add svs-vamana to info telemetry

* revert vamana (split to other PR)

* revert manually

* move vector overhead to index memory usage function (WIP)

* remove leftover

* fix vector overhead usage

(cherry picked from commit 870e099)
@codecov
Copy link

codecov bot commented Aug 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.40%. Comparing base (dba8dd0) to head (be8364e).
⚠️ Report is 7 commits behind head on 8.2.

Additional details and impacted files
@@            Coverage Diff             @@
##              8.2    #6641      +/-   ##
==========================================
- Coverage   89.46%   89.40%   -0.06%     
==========================================
  Files         253      253              
  Lines       41613    41615       +2     
  Branches     3725     3725              
==========================================
- Hits        37228    37206      -22     
- Misses       4336     4360      +24     
  Partials       49       49              
Flag Coverage Δ
flow 81.98% <90.00%> (-0.18%) ⬇️
unit 47.20% <30.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alonre24 alonre24 requested a review from meiravgri August 13, 2025 12:00
@alonre24 alonre24 changed the title Update vectors memory in total index info memory (#6613) Update vectors memory in total index info memory [8.2] [MOD-10841] Aug 13, 2025
@alonre24 alonre24 changed the title Update vectors memory in total index info memory [8.2] [MOD-10841] [8.2]Update vectors memory in total index info memory [MOD-10841] Aug 14, 2025
@alonre24 alonre24 changed the title [8.2]Update vectors memory in total index info memory [MOD-10841] [8.2] Update vectors memory in total index info memory [MOD-10841] Aug 14, 2025
@alonre24 alonre24 added this pull request to the merge queue Aug 14, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 14, 2025
@meiravgri meiravgri added this pull request to the merge queue Aug 15, 2025
Merged via the queue into 8.2 with commit 2ae320f Aug 15, 2025
17 checks passed
@meiravgri meiravgri deleted the backport-6613-to-8.2 branch August 15, 2025 06:21
YaacovHazan pushed a commit to redis/redis that referenced this pull request Sep 28, 2025
$ gitlog v8.2.1..v8.2.5

RediSearch/RediSearch#6867 Improve BUILD_INTEL_SVS_OPT flag validation
RediSearch/RediSearch#6845 Fix vector compression type reporting and
rename SVS environment variable
RediSearch/RediSearch#6794 Fix rpcountFree casting
RediSearch/RediSearch#6825 Bump vecsim 8.2.4
RediSearch/RediSearch#6787 Fix deadlock while RDB loading and RM_Yield
(#6763)
RediSearch/RediSearch#6723 Fix ACLUserMayAccessIndex
RediSearch/RediSearch#6701 Fix fd leak when OOM
RediSearch/RediSearch#6676 bump svs version
RediSearch/RediSearch#6671 Add SVS-VAMANA index to info fields telemetry
RediSearch/RediSearch#6665 Optimize rs_wall_clock_diff_ns by removing
redundant if branch
RediSearch/RediSearch#6641 Update vectors memory in total index info
memory
RediSearch/RediSearch#6634 Log DocTable capacity growth for memory
diagnostics
RediSearch/RediSearch#6649 Bump minimal redis version
RediSearch/RediSearch#6648 Fix "has map" performance
RediSearch/RediSearch#6645 Trie: hide expensive memory usage computation
behind a test-specific feature flag
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants