Skip to content

feat(mempool): Optimize mempool cleaning#1526

Merged
msbrogli merged 1 commit intomasterfrom
feat/optimize-mempool-cleaning
Jan 8, 2026
Merged

feat(mempool): Optimize mempool cleaning#1526
msbrogli merged 1 commit intomasterfrom
feat/optimize-mempool-cleaning

Conversation

@msbrogli
Copy link
Member

@msbrogli msbrogli commented Dec 20, 2025

Motivation

The changes in ./hathor in this PR have already been released in #1524 in the following commit: d929a7c

Acceptance Criteria

  • Include here all things that this PR should solve

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@msbrogli msbrogli requested a review from jansegre as a code owner December 20, 2025 11:31
@jansegre jansegre mentioned this pull request Dec 20, 2025
2 tasks
@msbrogli msbrogli force-pushed the feat/optimize-mempool-cleaning branch from 7b41aff to f1d7d02 Compare December 20, 2025 11:34
@github-actions
Copy link

github-actions bot commented Dec 20, 2025

🐰 Bencher Report

Branchfeat/optimize-mempool-cleaning
Testbedubuntu-22.04
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
minutes (m)
(Result Δ%)
Lower Boundary
minutes (m)
(Limit %)
Upper Boundary
minutes (m)
(Limit %)
sync-v2 (up to 20000 blocks)📈 view plot
🚷 view threshold
1.68 m
(-2.92%)Baseline: 1.73 m
1.55 m
(92.71%)
2.07 m
(80.90%)
🐰 View full continuous benchmarking report in Bencher

@msbrogli msbrogli force-pushed the feat/optimize-mempool-cleaning branch 2 times, most recently from 1eb7d64 to 3abd94a Compare December 20, 2025 11:54
@codecov
Copy link

codecov bot commented Dec 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.27%. Comparing base (e6f4d00) to head (cafc16b).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1526      +/-   ##
==========================================
+ Coverage   86.25%   86.27%   +0.02%     
==========================================
  Files         438      438              
  Lines       33736    33736              
  Branches     5267     5267              
==========================================
+ Hits        29100    29107       +7     
+ Misses       3622     3617       -5     
+ Partials     1014     1012       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jansegre jansegre mentioned this pull request Dec 22, 2025
2 tasks
@msbrogli msbrogli self-assigned this Jan 2, 2026
@msbrogli msbrogli moved this from Todo to In Progress (Done) in Hathor Network Jan 2, 2026
jansegre
jansegre previously approved these changes Jan 6, 2026
@glevco glevco moved this from In Progress (Done) to In Review (WIP) in Hathor Network Jan 6, 2026
@jansegre jansegre requested a review from glevco January 7, 2026 15:33
@msbrogli msbrogli force-pushed the feat/optimize-mempool-cleaning branch from 3abd94a to 56bb95b Compare January 7, 2026 15:42
@glevco glevco moved this from In Review (WIP) to In Review (Done) in Hathor Network Jan 8, 2026
@msbrogli msbrogli force-pushed the feat/optimize-mempool-cleaning branch from e6f4d00 to cafc16b Compare January 8, 2026 16:43
@msbrogli msbrogli merged commit cafc16b into master Jan 8, 2026
10 of 11 checks passed
@msbrogli msbrogli deleted the feat/optimize-mempool-cleaning branch January 8, 2026 16:43
@github-project-automation github-project-automation bot moved this from In Review (Done) to Waiting to be deployed in Hathor Network Jan 8, 2026
@jansegre jansegre mentioned this pull request Jan 14, 2026
2 tasks
@jansegre jansegre moved this from Waiting to be deployed to Done in Hathor Network Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants