Skip to content

[MOD-6056] Rename FT.PROFILE counter fields#7303

Merged
meiravgri merged 1 commit intomasterfrom
meiravg_rename_profile_fields
Nov 12, 2025
Merged

[MOD-6056] Rename FT.PROFILE counter fields#7303
meiravgri merged 1 commit intomasterfrom
meiravg_rename_profile_fields

Conversation

@meiravgri
Copy link
Collaborator

@meiravgri meiravgri commented Nov 11, 2025

Rename FT.PROFILE fields for clarity

Iterator fields:

  • CounterNumber of reading operations - counts read/skipTo operations performed by the iterator
  • SizeEstimated number of matches - upper-bound estimate of results the iterator will yield

Result Processor fields:

  • CounterResults processed - counts how many results passed through the processor

iterators `Size` -> `Estimated number of matches`
iterators `Counter` -> `Number of reading operations`
RP `Counter` -> 'Results processed'
@meiravgri meiravgri changed the title [MOD-6056] rename profile counters fields [MOD-6056] Rename FT.PROFILE counters fields Nov 11, 2025
@meiravgri meiravgri changed the title [MOD-6056] Rename FT.PROFILE counters fields [MOD-6056] Rename FT.PROFILE counter fields Nov 11, 2025
@codecov
Copy link

codecov bot commented Nov 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.10%. Comparing base (cdae5af) to head (aaf6998).
⚠️ Report is 42 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7303      +/-   ##
==========================================
+ Coverage   85.04%   85.10%   +0.06%     
==========================================
  Files         341      343       +2     
  Lines       51670    52912    +1242     
  Branches    12526    13803    +1277     
==========================================
+ Hits        43941    45029    +1088     
- Misses       7537     7688     +151     
- Partials      192      195       +3     
Flag Coverage Δ
flow 84.40% <100.00%> (+0.11%) ⬆️
unit 52.60% <0.00%> (+1.04%) ⬆️

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.

@meiravgri meiravgri requested a review from alonre24 November 11, 2025 11:32
@meiravgri meiravgri added this pull request to the merge queue Nov 11, 2025
Merged via the queue into master with commit 508087c Nov 12, 2025
25 checks passed
@meiravgri meiravgri deleted the meiravg_rename_profile_fields branch November 12, 2025 01:28
@redisearch-backport-pull-request
Copy link
Contributor

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

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

git fetch origin 8.2
git worktree add -d .worktree/backport-7303-to-8.2 origin/8.2
cd .worktree/backport-7303-to-8.2
git switch --create backport-7303-to-8.2
git cherry-pick -x 508087c62127c7cedb60cff55a4635e58dc76952

redisearch-backport-pull-request bot pushed a commit that referenced this pull request Nov 12, 2025
rename profile counters fields
iterators `Size` -> `Estimated number of matches`
iterators `Counter` -> `Number of reading operations`
RP `Counter` -> 'Results processed'

(cherry picked from commit 508087c)
@redisearch-backport-pull-request
Copy link
Contributor

Successfully created backport PR for 8.4:

meiravgri added a commit that referenced this pull request Nov 12, 2025
rename profile counters fields
iterators `Size` -> `Estimated number of matches`
iterators `Counter` -> `Number of reading operations`
RP `Counter` -> 'Results processed'

(cherry picked from commit 508087c)
github-merge-queue bot pushed a commit that referenced this pull request Nov 12, 2025
[MOD-6056] Rename `FT.PROFILE` counter fields (#7303)

rename profile counters fields
iterators `Size` -> `Estimated number of matches`
iterators `Counter` -> `Number of reading operations`
RP `Counter` -> 'Results processed'

(cherry picked from commit 508087c)

Co-authored-by: meiravgri <[email protected]>
github-merge-queue bot pushed a commit that referenced this pull request Nov 13, 2025
* [MOD-6056] Rename `FT.PROFILE` counter fields (#7303)

rename profile counters fields
iterators `Size` -> `Estimated number of matches`
iterators `Counter` -> `Number of reading operations`
RP `Counter` -> 'Results processed'

(cherry picked from commit 508087c)

* fix
@meiravgri
Copy link
Collaborator Author

/backport

@redisearch-backport-pull-request
Copy link
Contributor

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

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

git fetch origin 2.8
git worktree add -d .worktree/backport-7303-to-2.8 origin/2.8
cd .worktree/backport-7303-to-2.8
git switch --create backport-7303-to-2.8
git cherry-pick -x 508087c62127c7cedb60cff55a4635e58dc76952

@redisearch-backport-pull-request
Copy link
Contributor

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

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

git fetch origin 2.10
git worktree add -d .worktree/backport-7303-to-2.10 origin/2.10
cd .worktree/backport-7303-to-2.10
git switch --create backport-7303-to-2.10
git cherry-pick -x 508087c62127c7cedb60cff55a4635e58dc76952

@redisearch-backport-pull-request
Copy link
Contributor

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

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

git fetch origin 8.2
git worktree add -d .worktree/backport-7303-to-8.2 origin/8.2
cd .worktree/backport-7303-to-8.2
git switch --create backport-7303-to-8.2
git cherry-pick -x 508087c62127c7cedb60cff55a4635e58dc76952

@redisearch-backport-pull-request
Copy link
Contributor

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

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

git fetch origin 8.4
git worktree add -d .worktree/backport-7303-to-8.4 origin/8.4
cd .worktree/backport-7303-to-8.4
git switch --create backport-7303-to-8.4
git cherry-pick -x 508087c62127c7cedb60cff55a4635e58dc76952

meiravgri added a commit that referenced this pull request Nov 17, 2025
rename profile counters fields
iterators `Size` -> `Estimated number of matches`
iterators `Counter` -> `Number of reading operations`
RP `Counter` -> 'Results processed'

(cherry picked from commit 508087c)
meiravgri added a commit that referenced this pull request Nov 17, 2025
* [MOD-6056] Rename `FT.PROFILE` counter fields (#7303)

rename profile counters fields
iterators `Size` -> `Estimated number of matches`
iterators `Counter` -> `Number of reading operations`
RP `Counter` -> 'Results processed'

(cherry picked from commit 508087c)

* fix

(cherry picked from commit c65cc38)
github-merge-queue bot pushed a commit that referenced this pull request Nov 17, 2025
* [MOD-6056] Rename `FT.PROFILE` counter fields (#7303)

rename profile counters fields
iterators `Size` -> `Estimated number of matches`
iterators `Counter` -> `Number of reading operations`
RP `Counter` -> 'Results processed'

(cherry picked from commit 508087c)

* fix

* fix test_issues

* fix test_resp3
github-merge-queue bot pushed a commit that referenced this pull request Nov 17, 2025
[8.2] [MOD-6056] Rename `FT.PROFILE` counter fields  (#7341)

* [MOD-6056] Rename `FT.PROFILE` counter fields (#7303)

rename profile counters fields
iterators `Size` -> `Estimated number of matches`
iterators `Counter` -> `Number of reading operations`
RP `Counter` -> 'Results processed'

(cherry picked from commit 508087c)

* fix

(cherry picked from commit c65cc38)
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