Skip to content

[8.4] [RED-169833] fix: fix reporting used memory as unsigned long long#7276

Merged
JoanFM merged 2 commits into8.4from
backport-6971-to-8.4
Nov 10, 2025
Merged

[8.4] [RED-169833] fix: fix reporting used memory as unsigned long long#7276
JoanFM merged 2 commits into8.4from
backport-6971-to-8.4

Conversation

@redisearch-backport-pull-request
Copy link
Contributor

@redisearch-backport-pull-request redisearch-backport-pull-request bot commented Nov 10, 2025

Description

Backport of #6971 to 8.4.


Note

Switch INFO output of memory, GC, and error counters from double to unsigned long long.

  • INFO output type corrections:
    • Memory: Report used_memory_indexes, smallest_memory_index, largest_memory_index, and used_memory_vector_index as ULongLong (keep human-readable fields as doubles).
    • Garbage Collector: Report gc_bytes_collected, gc_total_cycles, gc_total_ms_run as ULongLong.
    • Warnings and Errors: Report errors_indexing_failures, errors_for_index_with_max_failures, OOM_indexing_failures_indexes_count as ULongLong.

Written by Cursor Bugbot for commit 4318052. This will update automatically on new commits. Configure here.

)

* fix: fix reporting used memory as unsigned long long

* change the bytes collected type to double again after change by other PR

(cherry picked from commit ca5b662)
@JoanFM JoanFM requested a review from oshadmi November 10, 2025 10:11
@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.64%. Comparing base (8266066) to head (4318052).
⚠️ Report is 2 commits behind head on 8.4.

Additional details and impacted files
@@            Coverage Diff             @@
##              8.4    #7276      +/-   ##
==========================================
- Coverage   85.65%   85.64%   -0.02%     
==========================================
  Files         326      326              
  Lines       50942    50942              
  Branches    10921    10921              
==========================================
- Hits        43636    43628       -8     
- Misses       7140     7148       +8     
  Partials      166      166              
Flag Coverage Δ
flow 84.18% <100.00%> (-0.13%) ⬇️
unit 51.73% <0.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.

@redisearch-backport-pull-request redisearch-backport-pull-request bot added this pull request to the merge queue Nov 10, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 10, 2025
@JoanFM JoanFM added this pull request to the merge queue Nov 10, 2025
Merged via the queue into 8.4 with commit 5d3e458 Nov 10, 2025
19 checks passed
@JoanFM JoanFM deleted the backport-6971-to-8.4 branch November 10, 2025 18:19
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