Skip to content

Conversation

@wlawt
Copy link
Contributor

@wlawt wlawt commented Jan 14, 2026

Closes #417

  • Keeps the current "e2e" inclusion metric (i.e., start tracking the moment we see it in the mpool)
  • Introduces "reth_transaction_tracing_transaction_inclusion_duration" which measures the time starting when a tx is marked as pending
  • Added unit tests

Next up (in follow up PR)

  • Make tracker Flashblocks aware

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Jan 14, 2026

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@wlawt wlawt marked this pull request as ready for review January 14, 2026 15:33
Copy link
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.

lgtm

@danyalprout
Copy link
Collaborator

@wlawt would you like to backport this to the releases/v0.3.0 branch? That way it'll get deployed fairly quickly. Should be pretty low risk too.

@danyalprout danyalprout added this pull request to the merge queue Jan 14, 2026
Merged via the queue into main with commit 2e44592 Jan 14, 2026
18 checks passed
@danyalprout danyalprout deleted the tracker-pending branch January 14, 2026 16:10
@wlawt wlawt added this to the v0.3.0 milestone Jan 14, 2026
wlawt added a commit that referenced this pull request Jan 14, 2026
danyalprout pushed a commit that referenced this pull request Jan 14, 2026
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.

feat(txpool): redefine tx inclusion in tx tracker

4 participants