FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Optimize client stats collection by pavel-ptashyts · Pull Request #2270 · AsyncHttpClient/async-http-client · GitHub

Optimize client stats collection - #2270

Merged
hyperxpro merged 1 commit into
AsyncHttpClient:mainfrom
maygemdev:perf/client-stats-single-pass
Jul 23, 2026
Merged

Optimize client stats collection#2270
hyperxpro merged 1 commit into
AsyncHttpClient:mainfrom
maygemdev:perf/client-stats-single-pass

Conversation

pavel-ptashyts commented Jul 20, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Replace stream collectors in ChannelManager.getClientStats() with explicit per-host counters.
  • Count idle pooled channels through direct partition iteration while preserving host filtering and tombstone-skipping behavior.
  • Add coverage for per-host idle counts so claimed channels are not reported as leasable.

Validation

  • ./mvnw -pl client -Dtest=DefaultChannelPoolTest,ClientStatsTest test
  • ./mvnw -pl client -DskipTests -Dgpg.skip=true verify
  • git diff --check

Attribution

Codex on behalf of Pavel Ptashyts

Replace stream grouping in client stats snapshots with explicit
per-host counters to avoid redundant collector passes when metrics are
scraped frequently. Count idle pooled channels with direct partition
iteration while preserving existing host filtering and tombstone
skipping behavior.

Add coverage for idle per-host counts so claimed channels are not
reported as leasable.

Codex on behalf of Pavel Ptashyts

Co-Authored-By: Codex <codex@openai.com>
pavel-ptashyts force-pushed the perf/client-stats-single-pass branch from 3bae6af to 50dbb79 Compare July 20, 2026 08:50
hyperxpro merged commit 903003b into AsyncHttpClient:main Jul 23, 2026
13 checks passed
pavel-ptashyts deleted the perf/client-stats-single-pass branch July 24, 2026 05:13
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL