Skip to content

Fix TTL dictionary - [MOD-10240]#6364

Merged
GuyAv46 merged 3 commits intomasterfrom
guyav-fix_ttl
Jun 24, 2025
Merged

Fix TTL dictionary - [MOD-10240]#6364
GuyAv46 merged 3 commits intomasterfrom
guyav-fix_ttl

Conversation

@GuyAv46
Copy link
Copy Markdown
Collaborator

@GuyAv46 GuyAv46 commented Jun 23, 2025

Describe the changes in the pull request

  • Remove redundant implementations
  • Fix (by removing) the incorrect comparison function, which should return true (1) on equality, and false (0) otherwise

Note

The bug can occur if a low ID is mapped to the same bucket as a high ID that is in the dictionary. Looking for the low ID may return the high ID as a match

Mark if applicable

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

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.92%. Comparing base (2dfefe9) to head (5344fe9).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6364      +/-   ##
==========================================
- Coverage   88.98%   88.92%   -0.07%     
==========================================
  Files         250      250              
  Lines       41415    41409       -6     
  Branches     3556     3556              
==========================================
- Hits        36854    36823      -31     
- Misses       4512     4537      +25     
  Partials       49       49              
Flag Coverage Δ
flow 81.35% <100.00%> (-0.18%) ⬇️
unit 47.04% <100.00%> (-0.02%) ⬇️

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.

@GuyAv46 GuyAv46 added the bug label Jun 23, 2025
@GuyAv46 GuyAv46 enabled auto-merge June 23, 2025 18:43
@GuyAv46 GuyAv46 added this pull request to the merge queue Jun 24, 2025
@GuyAv46 GuyAv46 changed the title Fix TTL dictionary Fix TTL dictionary - [MOD-10240] Jun 24, 2025
Merged via the queue into master with commit 5da8717 Jun 24, 2025
18 checks passed
@GuyAv46 GuyAv46 deleted the guyav-fix_ttl branch June 24, 2025 09:53
@redisearch-backport-pull-request

This comment was marked as resolved.

@redisearch-backport-pull-request
Copy link
Copy Markdown
Contributor

Backport failed for 8.0, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin 8.0
git worktree add -d .worktree/backport-6364-to-8.0 origin/8.0
cd .worktree/backport-6364-to-8.0
git switch --create backport-6364-to-8.0
git cherry-pick -x 5da87174a9eb6847485c375d973c3cb31e0befd9

GuyAv46 added a commit that referenced this pull request Jun 24, 2025
* fix TTL dictionary

* add a failing test

* delete dummy test

(cherry picked from commit 5da8717)
github-merge-queue bot pushed a commit that referenced this pull request Jun 24, 2025
Fix TTL dictionary - [MOD-10240] (#6364)

* fix TTL dictionary

* add a failing test

* delete dummy test

(cherry picked from commit 5da8717)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants