Skip to content

[2.10] [MOD-13431] Add logs to index commands#8065

Merged
lerman25 merged 1 commit into2.10from
backport-8036-to-2.10
Jan 18, 2026
Merged

[2.10] [MOD-13431] Add logs to index commands#8065
lerman25 merged 1 commit into2.10from
backport-8036-to-2.10

Conversation

@lerman25
Copy link
Collaborator

@lerman25 lerman25 commented Jan 18, 2026

backport #8036 to 2.10


Note

Adds notice-level logging around index lifecycle operations and ensures safe logging of names.

  • Log successful FT.CREATE, FT.DROP/FT.DROPINDEX, and FT.ALTER via RedisModule_Log using IndexSpec_FormatName(..., RSGlobalConfig.hideUserDataFromLog)
  • In drop flow, dup the formatted index name before freeing the spec and free it after logging
  • No changes to replication or command semantics

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

@lerman25 lerman25 requested a review from alonre24 January 18, 2026 09:11
@lerman25 lerman25 changed the title [MOD-13431] Add logs to index commands (#8036) [MOD-13431] Add logs to index commands Jan 18, 2026
@lerman25 lerman25 changed the title [MOD-13431] Add logs to index commands [2.10] [MOD-13431] Add logs to index commands Jan 18, 2026
@codecov
Copy link

codecov bot commented Jan 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.45%. Comparing base (07f7f39) to head (391fef7).
⚠️ Report is 1 commits behind head on 2.10.

Additional details and impacted files
@@            Coverage Diff             @@
##             2.10    #8065      +/-   ##
==========================================
- Coverage   89.48%   89.45%   -0.03%     
==========================================
  Files         210      210              
  Lines       36261    36268       +7     
==========================================
- Hits        32447    32444       -3     
- Misses       3814     3824      +10     
Flag Coverage Δ
flow 84.13% <100.00%> (-0.15%) ⬇️
unit 42.22% <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.

@lerman25 lerman25 added this pull request to the merge queue Jan 18, 2026
Merged via the queue into 2.10 with commit 291ee49 Jan 18, 2026
17 checks passed
@lerman25 lerman25 deleted the backport-8036-to-2.10 branch January 18, 2026 12:01
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