Skip to content

[metrics] track tx lifecycle time#75

Merged
wlawt merged 14 commits intomainfrom
tx-tracing
Sep 9, 2025
Merged

[metrics] track tx lifecycle time#75
wlawt merged 14 commits intomainfrom
tx-tracing

Conversation

@wlawt
Copy link
Copy Markdown
Contributor

@wlawt wlawt commented Sep 2, 2025

This PR adds an ExEx to track the lifecycle of a transaction.

Specifically, it tracks the time to inclusion when a transaction is first detected in the mempool to when it's included in a block. It also tracks the number of transactions that are dropped/replaced. These are recorded as Histogram metrics.

TODO:

  • Time to inclusion
  • # of transactions dropped
  • # of transactions replaced

@wlawt wlawt changed the title [wip][experiment] track tx lifecycle time [experiment] track tx lifecycle time Sep 8, 2025
@wlawt wlawt marked this pull request as ready for review September 8, 2025 19:03
Copy link
Copy Markdown
Collaborator

@danyalprout danyalprout left a comment

Choose a reason for hiding this comment

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

Looks great, left a few comments. Can we also downgrade all logs to debug from info. Thanks

@wlawt wlawt changed the title [experiment] track tx lifecycle time [metrics][experiment] track tx lifecycle time Sep 9, 2025
@wlawt wlawt changed the title [metrics][experiment] track tx lifecycle time [metrics] track tx lifecycle time Sep 9, 2025
@wlawt wlawt force-pushed the tx-tracing branch 2 times, most recently from 8929090 to 4e2af78 Compare September 9, 2025 15:18
@github-project-automation github-project-automation bot moved this to In Review in Project Tracking Sep 9, 2025
@wlawt wlawt merged commit a4c6b8a into main Sep 9, 2025
7 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in Project Tracking Sep 9, 2025
0xsimulacra pushed a commit to 0xsimulacra/base-node-reth that referenced this pull request Oct 1, 2025
* wip exex

* first pass

* remove readme

* track num dropped/replaced

* update toml structure

* naming

* nits

* use lru over hashmap

* record metrics

* always install exex

* fix ci

* combine inclusion and event fn together

* use install if

* fix ci
@danyalprout danyalprout deleted the tx-tracing branch January 9, 2026 18:47
refcell added a commit that referenced this pull request Feb 18, 2026
* chore(protocol): introduce other op block types

* fix: remove revision patches

* fix: bump msrv

* fix: remove block types

* fix: remove block types

* fix: remove block types
refcell added a commit that referenced this pull request Feb 18, 2026
* chore(protocol): introduce other op block types

* fix: remove revision patches

* fix: bump msrv

* fix: remove block types

* fix: remove block types

* fix: remove block types
danyalprout pushed a commit that referenced this pull request Mar 5, 2026
* chore(cgt): remove current spec

* feat(cgt): related specs

Signed-off-by: Hex <165055168+hexshire@users.noreply.github.com>
Co-authored-by: AgusDuha <81362284+agusduha@users.noreply.github.com>
Co-authored-by: Joxes <91908708+Joxess@users.noreply.github.com>

* fix: run linter

* chore: add deauthorize function

* chore: address review comments

* chore: cgt review comments 2

Signed-off-by: Hex <165055168+hexshire@users.noreply.github.com>
Co-authored-by: Joxes <91908708+Joxess@users.noreply.github.com>

* chore(cgt): run checks and linters

* chore: add deauthorize details

* chore: add missing words

* fix: update with the latest features

* fix: comments

* fix: comments

* chore: run linters

* fix: cgt flag (#74)

* feat: remove fund func (#75)

* feat: liquidity event (#76)

* feat: remove fund func

* fix: liquidity events

* fix: donate eth (#73)

* fix: donate eth

* fix: comments

* fix: donate eth

* chore: run linter

* chore: fix ci warning

* chore: fix words file

---------

Signed-off-by: Hex <165055168+hexshire@users.noreply.github.com>
Signed-off-by: Ashitaka <96790496+ashitakah@users.noreply.github.com>
Co-authored-by: AgusDuha <81362284+agusduha@users.noreply.github.com>
Co-authored-by: Joxes <91908708+Joxess@users.noreply.github.com>
Co-authored-by: Ashitaka <ashitaka@defi.sucks>
Co-authored-by: Ashitaka <96790496+ashitakah@users.noreply.github.com>
haardikk21 pushed a commit that referenced this pull request Mar 17, 2026
* wip exex

* first pass

* remove readme

* track num dropped/replaced

* update toml structure

* naming

* nits

* use lru over hashmap

* record metrics

* always install exex

* fix ci

* combine inclusion and event fn together

* use install if

* fix ci
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