Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

debug: unmarshal and print transaction distribution #5008

Merged
merged 2 commits into from
Jan 19, 2023

Conversation

cce
Copy link
Contributor

@cce cce commented Jan 11, 2023

Summary

Support for parsing and printing the transaction processing time histogram buckets in an easier-to-read format.

Test Plan

Updated tests in metrics_test

@codecov
Copy link

codecov bot commented Jan 11, 2023

Codecov Report

Merging #5008 (33e4286) into master (ecdcbf9) will increase coverage by 0.03%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master    #5008      +/-   ##
==========================================
+ Coverage   53.63%   53.67%   +0.03%     
==========================================
  Files         432      432              
  Lines       54057    54093      +36     
==========================================
+ Hits        28996    29033      +37     
+ Misses      22818    22811       -7     
- Partials     2243     2249       +6     
Impacted Files Coverage Δ
logging/telemetryspec/metric.go 81.91% <83.33%> (+0.88%) ⬆️
crypto/merkletrie/trie.go 66.42% <0.00%> (-2.19%) ⬇️
agreement/cryptoVerifier.go 67.60% <0.00%> (-2.12%) ⬇️
agreement/proposalManager.go 96.07% <0.00%> (-1.97%) ⬇️
crypto/merkletrie/node.go 91.62% <0.00%> (-1.87%) ⬇️
catchup/peerSelector.go 98.95% <0.00%> (-1.05%) ⬇️
data/transactions/verify/txn.go 70.24% <0.00%> (-0.49%) ⬇️
network/wsNetwork.go 64.92% <0.00%> (+0.17%) ⬆️
catchup/service.go 69.80% <0.00%> (+0.48%) ⬆️
network/wsPeer.go 69.03% <0.00%> (+2.60%) ⬆️
... and 2 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@cce cce marked this pull request as ready for review January 13, 2023 16:48
@algorandskiy algorandskiy merged commit 2aa2c91 into algorand:master Jan 19, 2023
@cce cce deleted the print-processing-distributions branch February 2, 2023 20:23
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