Skip to content

Conversation

@jannotti
Copy link
Contributor

@jannotti jannotti commented Dec 9, 2025

We used to report statistics on the number of transactions that expired in old blocks. We no longer do, but we were maintaining the large map required to do so.

Summary

Test Plan

We used to report statistics on the number of transactions that
expired in old blocks.  We no longer do, but we were maintaining the
large map required to do so.
@jannotti jannotti self-assigned this Dec 9, 2025
@codecov
Copy link

codecov bot commented Dec 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.68%. Comparing base (5031060) to head (446298c).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6513      +/-   ##
==========================================
- Coverage   47.86%   47.68%   -0.19%     
==========================================
  Files         662      655       -7     
  Lines       87945    87851      -94     
==========================================
- Hits        42099    41891     -208     
- Misses      43084    43184     +100     
- Partials     2762     2776      +14     

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

@jannotti jannotti marked this pull request as ready for review December 9, 2025 17:24
@jannotti jannotti requested review from cce and gmalouf December 9, 2025 17:25
@jannotti jannotti merged commit 3780647 into algorand:master Dec 9, 2025
40 checks passed
@jannotti jannotti deleted the remove-expiration-map branch December 9, 2025 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants