Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

polygon: fix bor_heimdall stage unwinding due to event processed blocks table #12148

Merged
merged 2 commits into from
Oct 1, 2024

Conversation

taratorio
Copy link
Member

@taratorio taratorio commented Oct 1, 2024

fixes

INFO] [09-30|16:59:20.466] aggregator unwind                        step=2940 txUnwindTo=4595202440 stepsRangeInDB="accounts:0.9, storage:0.9, code:0.9, commitment:0.0, receipt:0.9, logaddrs: 0.9, logtopics: 0.9, tracesfrom: 0.9, tracesto: 0.9"
[EROR] [09-30|16:59:20.671] Staged Sync                              err="[3/9 BorHeimdall] unexpected missing first processed block info entry when unwinding"

that table (BorEventProcessedBlocks) is only populated as part of astrid, so bor heimdall stage should not attempt to unwind it (same for BorProducerSelections table)

@taratorio taratorio changed the title Bor heimdall fix bor event processed blocks polygon: fix bor_heimdall stage unwinding due to event processed blocks table Oct 1, 2024
@Giulio2002 Giulio2002 enabled auto-merge (squash) October 1, 2024 11:49
@Giulio2002 Giulio2002 merged commit 2c1b97e into main Oct 1, 2024
9 checks passed
@Giulio2002 Giulio2002 deleted the bor-heimdall-fix-bor-event-processed-blocks branch October 1, 2024 12:15
taratorio added a commit that referenced this pull request Nov 5, 2024
This fixes this issue when running main with bor_heimdall stage:

```
[EROR] [10-31|09:13:03.527] Staged Sync                              err="[3/9 BorHeimdall] unexpected missing first processed block info entry when unwinding"
```

It was caused by a miss-merge after this PR:
#12148

---------

Co-authored-by: taratorio <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants