Skip to content

refactor: Rework http client pool#629

Merged
gtema merged 1 commit intomainfrom
client-pool
Mar 20, 2026
Merged

refactor: Rework http client pool#629
gtema merged 1 commit intomainfrom
client-pool

Conversation

@gtema
Copy link
Collaborator

@gtema gtema commented Mar 20, 2026

Using RwLock<HashMap<String, Arc>> inside the provider structure
makes compiler unhappy. Moving it into the separate structure allows
to half the crate compilation time.

Using RwLock<HashMap<String, Arc<Client>>> inside the provider structure
makes compiler unhappy. Moving it into the separate structure allows
to half the crate compilation time.
@github-actions
Copy link

🐰 Bencher Report

Branchclient-pool
Testbedubuntu-latest
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
nanoseconds (ns)
(Result Δ%)
Upper Boundary
nanoseconds (ns)
(Limit %)
fernet token/project📈 view plot
🚷 view threshold
1,435.50 ns
(-1.44%)Baseline: 1,456.48 ns
1,520.47 ns
(94.41%)
get_fernet_token_timestamp/project📈 view plot
🚷 view threshold
153.04 ns
(+1.14%)Baseline: 151.32 ns
172.76 ns
(88.59%)
🐰 View full continuous benchmarking report in Bencher

@github-actions
Copy link

🦢 Load Test Results

Goose Attack Report

Plan Overview

Action Started Stopped Elapsed Users
Increasing 26-03-20 11:26:31 26-03-20 11:26:33 00:00:02 0 → 4
Maintaining 26-03-20 11:26:33 26-03-20 11:27:03 00:00:30 4
Decreasing 26-03-20 11:27:03 26-03-20 11:27:03 00:00:00 0 ← 4

Request Metrics

Method Name # Requests # Fails Average (ms) Min (ms) Max (ms) RPS Failures/s
GET 7547 0 15.39 9 32 251.57 0.00
Aggregated 7547 0 15.39 9 32 251.57 0.00

Response Time Metrics

Method Name 50%ile (ms) 60%ile (ms) 70%ile (ms) 80%ile (ms) 90%ile (ms) 95%ile (ms) 99%ile (ms) 100%ile (ms)
GET 13 13 20 21 22 22 24 32
Aggregated 13 13 20 21 22 22 24 32

Status Code Metrics

Method Name Status Codes
GET 7,547 [200]
Aggregated 7,547 [200]

Transaction Metrics

Transaction # Times Run # Fails Average (ms) Min (ms) Max (ms) RPS Failures/s
ListUsers
0.0 0 0 0.00 0 0 0.00 0.00
0.1 4792 0 12.05 9 19 159.73 0.00
ValidateToken
1.0 0 0 0.00 0 0 0.00 0.00
1.1 2755 0 21.34 18 32 91.83 0.00
Aggregated 7547 0 15.39 9 32 251.57 0.00

Scenario Metrics

Transaction # Users # Times Run Average (ms) Min (ms) Max (ms) Scenarios/s Iterations
ListUsers 2 4790 12.05 9 19 159.67 2395.00
ValidateToken 2 2753 21.34 18 32 91.77 1376.50
Aggregated 4 7543 15.44 9 32 251.43 3771.50

Error Metrics

Method Name # Error

View full report

@gtema gtema merged commit 541d89b into main Mar 20, 2026
29 checks passed
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.

1 participant