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.

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

Fix leak due to cid queue never getting cleaned up #756

Merged
merged 3 commits into from
Dec 17, 2024

Conversation

gammazero
Copy link
Contributor

@gammazero gammazero commented Dec 16, 2024

This removes entries from the queue portion of CidQueue that were removed from the map portion.

Credit to @Wondertan for finding and proposing fix.

@gammazero gammazero requested a review from a team as a code owner December 16, 2024 18:34
This removes entries from the queue portion of CidQueue that were removed from the map portion.
@gammazero gammazero force-pushed the fix-leak-in-sessionwants branch from a36ffa2 to 8b25b78 Compare December 16, 2024 18:37
Copy link

codecov bot commented Dec 16, 2024

Codecov Report

Attention: Patch coverage is 96.42857% with 1 line in your changes missing coverage. Please review.

Project coverage is 60.55%. Comparing base (e11e4ed) to head (cdc19ae).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
bitswap/client/internal/session/sessionwants.go 92.30% 1 Missing ⚠️

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #756      +/-   ##
==========================================
+ Coverage   60.48%   60.55%   +0.07%     
==========================================
  Files         245      245              
  Lines       31102    31102              
==========================================
+ Hits        18813    18835      +22     
+ Misses      10621    10598      -23     
- Partials     1668     1669       +1     
Files with missing lines Coverage Δ
...ernal/blockpresencemanager/blockpresencemanager.go 96.80% <100.00%> (+3.40%) ⬆️
bitswap/client/internal/session/cidqueue.go 90.00% <100.00%> (+38.71%) ⬆️
bitswap/client/internal/session/sessionwants.go 89.56% <92.30%> (+0.57%) ⬆️

... and 12 files with indirect coverage changes

@gammazero gammazero requested a review from hsanjuan December 17, 2024 20:43
@gammazero
Copy link
Contributor Author

gammazero commented Dec 17, 2024

@hsanjuan I think my latest change satisfies what you were asking for. I will merge this PR in a bit unless you have any objections.

@gammazero gammazero merged commit 00f508d into main Dec 17, 2024
15 checks passed
@gammazero gammazero deleted the fix-leak-in-sessionwants branch December 17, 2024 20:53
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.

2 participants