Skip to content

fix(observability): do not account processed entries count when list none#7269

Merged
koushiro merged 2 commits intoapache:mainfrom
dentiny:hjiang/no-count-none-entry-count
Mar 24, 2026
Merged

fix(observability): do not account processed entries count when list none#7269
koushiro merged 2 commits intoapache:mainfrom
dentiny:hjiang/no-count-none-entry-count

Conversation

@dentiny
Copy link
Copy Markdown
Contributor

@dentiny dentiny commented Mar 23, 2026

Which issue does this PR close?

Closes #7268

Rationale for this change

As mentioned in the issue, I think when list operation returns None, it should be accounted as processed entries count.
But happy to discuss / close PR if maintainers have different opinions.

Are there any user-facing changes?

No

AI Usage Statement

No AI involvement

@dentiny dentiny requested a review from Xuanwo as a code owner March 23, 2026 23:55
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. releases-note/fix The PR fixes a bug or has a title that begins with "fix" labels Mar 23, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Mar 24, 2026
@koushiro koushiro merged commit bc6e2e4 into apache:main Mar 24, 2026
71 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/fix The PR fixes a bug or has a title that begins with "fix" size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: processed entries count accounts for "None" returned on list operation

2 participants