Skip to content

fix(evmreader): avoid delaying work by skiping old blocks notifications#781

Open
renatomaia wants to merge 1 commit into
feature/bump-contracts-3.0from
fix/HandleSlowProcessingOfNewBlocks
Open

fix(evmreader): avoid delaying work by skiping old blocks notifications#781
renatomaia wants to merge 1 commit into
feature/bump-contracts-3.0from
fix/HandleSlowProcessingOfNewBlocks

Conversation

@renatomaia

Copy link
Copy Markdown

EVM Reader was processing each single new chain block notification, therefore when the processing was too slow (>2 seconds) the notifications accumulated and the EVM Reader could not keep up since worked one block at time. The fix is to skip old block notifications when a more recent is already available.

@renatomaia renatomaia requested review from mpolitzer and vfusco May 27, 2026 12:31
@renatomaia renatomaia force-pushed the fix/HandleSlowProcessingOfNewBlocks branch 2 times, most recently from c323cc3 to 1baa9f1 Compare May 27, 2026 14:38
Comment thread internal/evmreader/evmreader.go Outdated
@renatomaia renatomaia marked this pull request as draft May 27, 2026 18:03
@renatomaia renatomaia force-pushed the fix/HandleSlowProcessingOfNewBlocks branch 3 times, most recently from ddd9f16 to 6961df9 Compare May 29, 2026 13:45
@renatomaia renatomaia changed the base branch from next/2.0 to feature/bump-contracts-3.0 May 29, 2026 14:12
@renatomaia renatomaia force-pushed the fix/HandleSlowProcessingOfNewBlocks branch from 6961df9 to a74c247 Compare May 29, 2026 15:35
@vfusco vfusco force-pushed the feature/bump-contracts-3.0 branch from 89141df to f16b445 Compare May 29, 2026 21:09
@renatomaia renatomaia force-pushed the fix/HandleSlowProcessingOfNewBlocks branch from a74c247 to f0d2e92 Compare June 1, 2026 12:12
@vfusco vfusco force-pushed the feature/bump-contracts-3.0 branch from f16b445 to 4c9401e Compare June 5, 2026 21:16
@renatomaia renatomaia force-pushed the fix/HandleSlowProcessingOfNewBlocks branch from f0d2e92 to 49b70a0 Compare June 8, 2026 17:14
@renatomaia renatomaia self-assigned this Jun 9, 2026
@renatomaia renatomaia marked this pull request as ready for review June 9, 2026 13:39
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