Skip to content

[2.10] [RED-169833] fix: fix reporting used memory as unsigned long long (#6…#7277

Merged
JoanFM merged 1 commit into2.10from
backport-6971-to-2.10
Nov 11, 2025
Merged

[2.10] [RED-169833] fix: fix reporting used memory as unsigned long long (#6…#7277
JoanFM merged 1 commit into2.10from
backport-6971-to-2.10

Conversation

@JoanFM
Copy link
Collaborator

@JoanFM JoanFM commented Nov 10, 2025

…971)

  • 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)

Describe the changes in the pull request

A clear and concise description of what the PR is solving, including:

  1. Current: The current state briefly
  2. Change: What is the change
  3. Outcome: Adding the outcome

Which additional issues this PR fixes

  1. MOD-...
  2. #...

Main objects this PR modified

  1. ...

Mark if applicable

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

Note

Switches several INFO metrics from double to unsigned long long across memory, GC, queries/errors reporting.

  • Info metrics type adjustments (src/info/info_redis.c):
    • Memory section: used_memory_indexes, smallest_memory_index, largest_memory_index, used_memory_vector_index now use ULongLong (human-readable fields remain doubles).
    • GC section: bytes_collected, total_cycles, total_ms_run switched to ULongLong.
    • Queries section: total_query_execution_time_ms confirmed as ULongLong alongside other counters.
    • Warnings and errors: errors_indexing_failures, errors_for_index_with_max_failures, OOM_indexing_failures_indexes_count switched to ULongLong.

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

@JoanFM JoanFM requested a review from oshadmi November 10, 2025 10:11
)

* 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 force-pushed the backport-6971-to-2.10 branch from 84f93d8 to d8e7387 Compare November 10, 2025 10:15
@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.24%. Comparing base (4c0afa3) to head (d8e7387).
⚠️ Report is 2 commits behind head on 2.10.

Additional details and impacted files
@@            Coverage Diff             @@
##             2.10    #7277      +/-   ##
==========================================
- Coverage   89.27%   89.24%   -0.03%     
==========================================
  Files         207      207              
  Lines       35309    35309              
==========================================
- Hits        31523    31513      -10     
- Misses       3786     3796      +10     
Flag Coverage Δ
flow 83.82% <100.00%> (-0.16%) ⬇️
unit 42.47% <0.00%> (ø)

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.

@JoanFM JoanFM self-assigned this Nov 10, 2025
@JoanFM JoanFM enabled auto-merge November 10, 2025 14:58
@JoanFM JoanFM added this pull request to the merge queue Nov 11, 2025
Merged via the queue into 2.10 with commit 0113e0e Nov 11, 2025
13 checks passed
@JoanFM JoanFM deleted the backport-6971-to-2.10 branch November 11, 2025 09:12
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