Skip to content

Handle stats error#1584

Merged
stefangutica merged 1 commit intodevelopmentfrom
try-catch-stats
Jan 26, 2026
Merged

Handle stats error#1584
stefangutica merged 1 commit intodevelopmentfrom
try-catch-stats

Conversation

@stefangutica
Copy link
Collaborator

Reasoning

  • raw error are returned to the user

Proposed Changes

  • catch error and log only a general message

How to test

  • raw error should be logged and a general error should be returned to the user

@github-actions
Copy link

k6 load testing comparison.
Base Commit Hash: 8149c3d
Target Commit Hash: c88c3d0

Metric Base Target Diff
AvgMax9095AvgMax9095AvgMax9095
Mex74.593788.02121.49162.3063.132923.96106.75122.85-15.36% ✅-22.81% ✅-12.14% ✅-24.31% ✅
Nodes1608.1126999.64525.8914481.37868.2023365.45259.001594.72-46.01% ✅-13.46% ✅-50.75% ✅-88.99% ✅
Accounts115.70695.33183.94272.5399.413793.65162.93211.77-14.08% ✅+445.59% 🔴-11.42% ✅-22.29% ✅
Tokens91.865141.95176.38237.5674.183785.85148.22185.22-19.24% ✅-26.37% ✅-15.97% ✅-22.03% ✅
Transactions127.371621.75225.70297.25109.943189.24174.15223.96-13.68% ✅+96.65% 🔴-22.84% ✅-24.65% ✅
Blocks113.293097.13182.17241.4195.912515.16152.37175.71-15.34% ✅-18.79% ✅-16.36% ✅-27.21% ✅
Pool75.375081.30117.44153.3662.942098.23107.59124.54-16.49% ✅-58.71% ✅-8.38% ✅-18.79% ✅
Test Run Duration60003.7160004.89

Legend: Avg - Average Response Time, Max - Maximum Response Time, 90 - 90th Percentile, 95 - 95th Percentile
All times are in milliseconds.

@stefangutica stefangutica merged commit 4d7881a into development Jan 26, 2026
6 checks passed
@stefangutica stefangutica deleted the try-catch-stats branch January 26, 2026 09:11
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.

3 participants