Skip to content

Fix memory leak in QuadTreeIndex#1679

Merged
andrey-kashcheev merged 1 commit intoheremaps:masterfrom
mzimonczyk-here:fix_memoryleak_in_quadtreeindex
Mar 11, 2026
Merged

Fix memory leak in QuadTreeIndex#1679
andrey-kashcheev merged 1 commit intoheremaps:masterfrom
mzimonczyk-here:fix_memoryleak_in_quadtreeindex

Conversation

@mzimonczyk-here
Copy link
Contributor

Thread local instance of MemoryPoolAllocator kept memory allocated for previous documents and grew during runtime.

@mzimonczyk-here mzimonczyk-here force-pushed the fix_memoryleak_in_quadtreeindex branch 2 times, most recently from a9f04f9 to b697a86 Compare March 10, 2026 13:34
@codecov
Copy link

codecov bot commented Mar 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.41%. Comparing base (3cfddaa) to head (26a36d3).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1679      +/-   ##
==========================================
- Coverage   80.42%   80.41%   -0.01%     
==========================================
  Files         351      351              
  Lines       14221    14219       -2     
  Branches     1544     1543       -1     
==========================================
- Hits        11436    11433       -3     
- Misses       2156     2158       +2     
+ Partials      629      628       -1     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mzimonczyk-here mzimonczyk-here force-pushed the fix_memoryleak_in_quadtreeindex branch from b697a86 to f5bdc37 Compare March 10, 2026 14:05
Thread local instance of MemoryPoolAllocator kept memory allocated
for previous documents and grew during runtime.

Relates-To: HNAV-13357
Signed-off-by: Michal Zimonczyk <ext-michal.zimonczyk@here.com>
@mzimonczyk-here mzimonczyk-here force-pushed the fix_memoryleak_in_quadtreeindex branch from f5bdc37 to 26a36d3 Compare March 10, 2026 14:42
@andrey-kashcheev andrey-kashcheev merged commit a3c7f31 into heremaps:master Mar 11, 2026
39 of 40 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.

3 participants