Skip to content

Supernode: Faster Processing with Backoff#19037

Merged
geoknee merged 1 commit intodevelopfrom
supernode/InteropRapidAdvance
Feb 9, 2026
Merged

Supernode: Faster Processing with Backoff#19037
geoknee merged 1 commit intodevelopfrom
supernode/InteropRapidAdvance

Conversation

@axelKingsley
Copy link
Contributor

@axelKingsley axelKingsley commented Jan 30, 2026

What

Adaptive tick rate for interop activity

  • Lowered tickerPeriod from 500ms → 50ms for fast progress when data is available
  • Added backoffPeriod (1s) when chains aren't ready, preventing busy-waiting
  • Extracted the error sleep into errorBackoffPeriod (2s) for clarity
  • progressAndRecord(): Now returns (verifiedAdvanced bool, err error) to signal whether the verified timestamp was advanced, driving the backoff decision
  • Unit tests now assert advancement

@axelKingsley axelKingsley requested a review from a team as a code owner January 30, 2026 22:44
@axelKingsley axelKingsley requested a review from nonsense January 30, 2026 22:44
@codecov
Copy link

codecov bot commented Jan 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.7%. Comparing base (bbb5143) to head (0761915).
⚠️ Report is 61 commits behind head on develop.

❗ There is a different number of reports uploaded between BASE (bbb5143) and HEAD (0761915). Click for more details.

HEAD has 7 uploads less than BASE
Flag BASE (bbb5143) HEAD (0761915)
cannon-go-tests-64 2 0
contracts-bedrock-tests 6 1
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #19037      +/-   ##
===========================================
- Coverage     76.3%    65.7%   -10.7%     
===========================================
  Files          188      133      -55     
  Lines        10937     6907    -4030     
===========================================
- Hits          8350     4538    -3812     
+ Misses        2441     2369      -72     
+ Partials       146        0     -146     
Flag Coverage Δ
cannon-go-tests-64 ?
contracts-bedrock-tests 65.7% <ø> (-16.0%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 79 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@axelKingsley axelKingsley requested review from geoknee and removed request for nonsense February 3, 2026 18:57
Copy link
Contributor

@karlfloersch karlfloersch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@geoknee geoknee added the A-op-supernode Area: op-supernode label Feb 5, 2026
@axelKingsley axelKingsley force-pushed the supernode/InteropRapidAdvance branch 2 times, most recently from 45fd7a6 to c88046f Compare February 6, 2026 22:16
@axelKingsley axelKingsley force-pushed the supernode/InteropRapidAdvance branch from c88046f to 653eb24 Compare February 6, 2026 22:18
@axelKingsley axelKingsley added this pull request to the merge queue Feb 9, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 9, 2026
@geoknee geoknee added this pull request to the merge queue Feb 9, 2026
Merged via the queue into develop with commit db87881 Feb 9, 2026
79 checks passed
@geoknee geoknee deleted the supernode/InteropRapidAdvance branch February 9, 2026 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-op-supernode Area: op-supernode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants