Skip to content

docs(tip-1077): add expiring nonce time wheel#5582

Draft
legion2002 wants to merge 2 commits into
mainfrom
tip/1077
Draft

docs(tip-1077): add expiring nonce time wheel#5582
legion2002 wants to merge 2 commits into
mainfrom
tip/1077

Conversation

@legion2002

Copy link
Copy Markdown
Contributor

Adds TIP-1077 for replacing the expiring nonce ring with a time-wheel replay table sized at 131,072 cells per bucket.

Documents the 256 MiB slot+value footprint, parameter tradeoffs, hardfork migration from the ring layout, and alternatives considered for cleanup and fixed-table designs.

Comment thread tips/tip-1077.md Outdated
state transition rules. A block containing a replay, an expired expiring nonce transaction, or a
transaction whose probe path is exhausted is invalid.

Offchain cleanup services are not trusted for replay safety. This TIP does not rely on any MEV,

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.

nit: Not sure this is really relevant for this TIP

Comment thread tips/tip-1077.md Outdated
Comment on lines +279 to +280
5. Transaction pool replay tracking MUST switch to sender-scoped replay hashes and/or time-wheel
cell matches instead of detecting inclusion by `expiringNonceSeen[hash]` storage writes.

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.

MUST switch to sender-scoped replay hashes

Replay detection already uses sender-scoped replay hashes, doesn't it? Afaiu this TIP isn't changing how the replay hash is being calculated.

MUST switch to sender-scoped replay hashes and/or time-wheel cell matches

This feels ambiguous to me, especially the "and/or" part... replay hashes in the time-wheel cells become the only replay detection mechanism, I'm not sure why there's an "or" in here.

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