Skip to content

[OOC] Add OOCPackedCache - #2534

Merged
janniklinde merged 3 commits into
apache:mainfrom
janniklinde:ooc/planner-merge-5-rebase
Jul 7, 2026
Merged

janniklinde merged 3 commits into
apache:mainfrom
janniklinde:ooc/planner-merge-5-rebase

Conversation

@janniklinde

Copy link
Copy Markdown
Contributor

This PR adds a new OOCCache implementation that supports packing small tiles together into one logical cache entry.

@codecov

codecov Bot commented Jul 6, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 63.09859% with 262 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.60%. Comparing base (afe793a) to head (3539bd9).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...sysds/runtime/ooc/cache/packed/OOCPackedCache.java 63.53% 105 Missing and 54 partials ⚠️
...sysds/runtime/ooc/cache/packed/PackedPinState.java 67.14% 35 Missing and 11 partials ⚠️
...he/sysds/runtime/ooc/cache/packed/PackedBlock.java 17.85% 23 Missing ⚠️
...he/sysds/runtime/ooc/cache/packed/PackBuilder.java 67.69% 15 Missing and 6 partials ⚠️
.../runtime/ooc/cache/packed/PackedCacheLocation.java 68.18% 4 Missing and 3 partials ⚠️
.../runtime/ooc/cache/io/SpillableObjectRegistry.java 0.00% 3 Missing ⚠️
...ds/runtime/ooc/cache/packed/PackedUnpinHandle.java 81.25% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2534      +/-   ##
============================================
+ Coverage     71.59%   71.60%   +0.01%     
- Complexity    49684    49851     +167     
============================================
  Files          1596     1602       +6     
  Lines        192298   193007     +709     
  Branches      37675    37787     +112     
============================================
+ Hits         137680   138209     +529     
- Misses        43909    44017     +108     
- Partials      10709    10781      +72     

☔ View full report in Codecov by Harness.
📢 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.

@janniklinde
janniklinde merged commit 108229b into apache:main Jul 7, 2026
49 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in SystemDS PR Queue Jul 7, 2026
Bruno-Ignaz pushed a commit to Bruno-Ignaz/systemds that referenced this pull request Sep 1, 2026
Adds a new OOCCache implementation that supports packing small tiles together into one logical cache entry.
@janniklinde
janniklinde deleted the ooc/planner-merge-5-rebase branch October 1, 2026 12:50
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.

1 participant