Skip to content

DEV: add warning to BM25STD scoring page#2989

Merged
dwdougherty merged 2 commits intomainfrom
DOC-6443
Apr 14, 2026
Merged

DEV: add warning to BM25STD scoring page#2989
dwdougherty merged 2 commits intomainfrom
DOC-6443

Conversation

@dwdougherty
Copy link
Copy Markdown
Collaborator

@dwdougherty dwdougherty commented Apr 7, 2026

Note

Low Risk
Low risk docs-only change that adds guidance about potentially unstable FT.SEARCH scores after document deletion/replacement until garbage collection runs.

Overview
Adds a warning to the BM25STD scorer documentation noting that deleting or replacing indexed documents can temporarily skew scores for other documents sharing terms, and that FT.SEARCH scores may remain inaccurate until the Redis Search garbage collector removes stale term data.

Reviewed by Cursor Bugbot for commit 4fe705d. Bugbot is set up for automated code reviews on this repo. Configure here.

@dwdougherty dwdougherty requested review from a team and oshadmi April 7, 2026 14:27
@dwdougherty dwdougherty self-assigned this Apr 7, 2026
@dwdougherty dwdougherty added the dev label Apr 7, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

DOC-6443

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

@jit-ci
Copy link
Copy Markdown

jit-ci bot commented Apr 7, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

Copy link
Copy Markdown
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dwdougherty
Copy link
Copy Markdown
Collaborator Author

Thank you, @andy-stark-redis!

@dwdougherty dwdougherty requested a review from ikalchev April 10, 2026 17:59
@dwdougherty
Copy link
Copy Markdown
Collaborator Author

Thank you, @ikalchev!

@dwdougherty dwdougherty merged commit 1ed530b into main Apr 14, 2026
47 checks passed
@dwdougherty dwdougherty deleted the DOC-6443 branch April 14, 2026 13:38
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.

3 participants