Skip to content

Conversation

@benaadams
Copy link
Member

Changes

  • Takes a significant time of ForkChoice and we put it into background in other places
image

Types of changes

What types of changes does your code introduce?

  • Optimization

Testing

Requires testing

  • No

Copy link
Member

@LukaszRozmej LukaszRozmej left a comment

Choose a reason for hiding this comment

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

  1. Consistency - bloom storage needs to be updated before we update the head/main chain so when query comes it is already updated.
  2. With @alexb5dh log index we will be retiring and removing bloom storage altogether after a while.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants