Skip to content

OverlayV2: Add metrics for tracking tx e2e latency - #5330

Merged
marta-lokhova merged 1 commit into
stellar:overlay-v2-sharedfrom
drebelsky:track-tx-latency
Jul 16, 2026
Merged

OverlayV2: Add metrics for tracking tx e2e latency#5330
marta-lokhova merged 1 commit into
stellar:overlay-v2-sharedfrom
drebelsky:track-tx-latency

Conversation

@drebelsky

Copy link
Copy Markdown
Contributor

Add per-node metrics for tracking tx e2e latency. The metrics are gated behind the config flag LOADGEN_MEASURE_TX_LATENCY_FOR_TESTING and are split into two kinds: per-run and cumulative. The per-run metrics are only updated at the successful completion of a loadgen run (so that there's one overall number we can use as a P50/75/99, etc...), while the cumulative numbers use a Medida timer (for more granular monitoring).

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@marta-lokhova marta-lokhova left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good overall! added a few questions and nits, mostly around perf

Comment thread src/ledger/LedgerManagerImpl.cpp
Comment thread src/main/Config.cpp Outdated
Comment thread src/ledger/LedgerManagerImpl.h
Comment thread src/ledger/LedgerManagerImpl.cpp Outdated
Comment thread src/ledger/LedgerManagerImpl.cpp

@marta-lokhova marta-lokhova left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Feel free to squash commits and we can merge.

@marta-lokhova
marta-lokhova merged commit dccd80f into stellar:overlay-v2-shared Jul 16, 2026
5 checks passed
pull Bot pushed a commit to Bloxxix/stellar-core that referenced this pull request Jul 20, 2026
Adds metrics for tracking e2e latency in testing. Same work as stellar#5330.
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