Skip to content

Conversation

@frisitano
Copy link
Collaborator

@frisitano frisitano commented Dec 22, 2025

Overview

This PR removes the cache in the watcher component and updates the processing logic of L1 events. L1 event processing is now done in groups. That is to say that given a sequence of logs, we split the logs into contiguous groups of logs with the same topic signature. These groups are then passed to the respective log handlers. Additionally, we remove the RPC cache as I suspect this was causing issues because the cache was stateful and therefore misbehaved during L1 reorgs - resulting in batch commits being produced with an incorrect blob versioned hash (empty). The rpc cache is less relevant for performance because we introduced a cache in the alloy provider in #439.

I have synced this on a VM.

@codspeed-hq
Copy link

codspeed-hq bot commented Dec 22, 2025

CodSpeed Performance Report

Merging #462 will not alter performance

Comparing refactor/watcher-refactor (1e67804) with main (c7f816e)

Summary

✅ 2 untouched

@frisitano frisitano merged commit c4927d6 into main Dec 23, 2025
15 checks passed
@frisitano frisitano deleted the refactor/watcher-refactor branch December 23, 2025 10:28
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