ChainSafe / ChainSafe/lodestar

Improve processAttestations time

Open
#8,621 1 comment 0 reactions 0 assignees View on GitHub
meta-bug
Dominant language
TypeScript
Stars
1.4k
Forks
483
Avg merge
1d 16h
Merged PRs (30d)
150

Description

### Describe the bug

most of the time to process a block is for `processAttestation`

Image

after electra, a validator could appear in multiple packed attestations, hence we should ignore the 2nd appearance onward

### Expected behavior

process attestation of a validator at most once per block

### Steps to reproduce

_No response_

### Additional context

_No response_

### Operating system

Linux

### Lodestar version or commit hash

v1.36.0

Contributor guide

Open the contributing guide

Research direction

Start at the `processAttestation` processing path identified in the issue and inspect how attestations are handled within a block, especially after Electra. Done means a validator appearing in multiple packed attestations is processed at most once per block.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
blockchain, performance
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.