Skip to content

added pubkey bidirectional cache#9345

Open
Fraccaman wants to merge 2 commits into
sigp:unstablefrom
Fraccaman:cache-builder-pubkey
Open

added pubkey bidirectional cache#9345
Fraccaman wants to merge 2 commits into
sigp:unstablefrom
Fraccaman:cache-builder-pubkey

Conversation

@Fraccaman
Copy link
Copy Markdown

@Fraccaman Fraccaman commented May 22, 2026

Issue Addressed

Closes #8783

Proposed Changes

This PR implements my understanding of how to fix the scenario described in #8783.
As described in the issue, this PR introduced a bidirectional cache between builder_index and public_key. It then leverage the new data structure to avoid the O(n) search.

Additional Info

  • the only function Im seeing that modifies the length of the builder list is add_builder_to_registry. Here, a new builder is added to the list. My assumption is that the builder list 1) never shrinks 2) builders pubkeys never change. This means that the cache can only miss some entries from the builder list but will never have more entries that the builder list.

Let me know if the logic is wrong or if anything should be modified :)

@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented May 22, 2026

CLA assistant check
All committers have signed the CLA.

@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented May 22, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

1 participant