Skip to content

Adjust SRC buffer from 500ms to 100ms#2179

Merged
cffls merged 1 commit into
developfrom
decrese_interrupt_reserve
Apr 1, 2026
Merged

Adjust SRC buffer from 500ms to 100ms#2179
cffls merged 1 commit into
developfrom
decrese_interrupt_reserve

Conversation

@cffls

@cffls cffls commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

Most of time, SRC is less than 100ms. For this reason, we should allow the worker to spend more time in transaction execution. This change will be crucial when we reduce the block time.

Most of time, SRC is less than 100ms. For this reason, we should allow the worker to spend more time in transaction execution. This change will be crucial when we reduce the block time.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@sonarqubecloud

sonarqubecloud Bot commented Apr 1, 2026

Copy link
Copy Markdown

@claude

claude Bot commented Apr 1, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@codecov

codecov Bot commented Apr 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 51.90%. Comparing base (a1c5b22) to head (54f4705).
⚠️ Report is 3 commits behind head on develop.

Files with missing lines Patch % Lines
miner/worker.go 50.00% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (50.00%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2179      +/-   ##
===========================================
- Coverage    51.92%   51.90%   -0.02%     
===========================================
  Files          884      884              
  Lines       155431   155431              
===========================================
- Hits         80701    80675      -26     
- Misses       69524    69543      +19     
- Partials      5206     5213       +7     
Files with missing lines Coverage Δ
miner/worker.go 72.49% <50.00%> (-0.13%) ⬇️

... and 17 files with indirect coverage changes

Files with missing lines Coverage Δ
miner/worker.go 72.49% <50.00%> (-0.13%) ⬇️

... and 17 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.

@cffls cffls merged commit e1ddb72 into develop Apr 1, 2026
23 of 26 checks passed
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.

4 participants