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.

Sign up for GitHub

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

Jump to bottom

NONEVM-1115 logpoller backfill #1012

Open
wants to merge 29 commits into
base: develop
Choose a base branch
from
Open

NONEVM-1115 logpoller backfill #1012

wants to merge 29 commits into from

Conversation

dhaidashenko
Copy link
Collaborator

@dhaidashenko dhaidashenko commented Jan 13, 2025 •
edited
Loading

Implemented main LogPoller loop.
Forward-looking block polling was replaced by global backfill to reduce the complexity and need to maintain two ways of fetching logs.
Draft of a performance test showed ~10x decrease in time needed to catchup from 10,000 slots lag on Solana Mainnet (assuming that CCIP uses a single contract for all interactions and that transaction frequency is similar to the top bridge available on Solana).

@dhaidashenko dhaidashenko requested review from a team as code owners January 13, 2025 15:12
@dhaidashenko dhaidashenko marked this pull request as draft January 13, 2025 15:12
@cl-sonarqube-production
Copy link

Quality Gate failed Quality Gate failed

Failed conditions
64.7% Coverage on New Code (required ≥ 75%)

See analysis details on SonarQube

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

@EasterTheBunny EasterTheBunny EasterTheBunny approved these changes

@reductionista reductionista reductionista approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dhaidashenko @reductionista @EasterTheBunny