Skip to content

[SYSTEMDS-3891] Add Generalized TSMMOOCPrimitive with Live Consumption - #2605

Merged
janniklinde merged 1 commit into
apache:mainfrom
janniklinde:ooc/w1
Sep 3, 2026
Merged

janniklinde merged 1 commit into
apache:mainfrom
janniklinde:ooc/w1

Conversation

@janniklinde

Copy link
Copy Markdown
Contributor

This patch introduces a planner-backed tsmm primitive for both live and fully materialized input streams.

Assisted-by: AI

@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 62.64775% with 158 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.39%. Comparing base (0a245ab) to head (f6f2cb6).
⚠️ Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
...sysds/runtime/ooc/primitives/TSMMOOCPrimitive.java 62.79% 100 Missing and 28 partials ⚠️
...untime/ooc/primitives/MaterializeOOCPrimitive.java 53.84% 2 Missing and 10 partials ⚠️
...time/ooc/store/IndexedMaterializedStoreReader.java 36.36% 6 Missing and 1 partial ⚠️
...che/sysds/runtime/ooc/store/MaterializedStore.java 73.91% 3 Missing and 3 partials ⚠️
...s/runtime/instructions/ooc/TSMMOOCInstruction.java 25.00% 2 Missing and 1 partial ⚠️
...che/sysds/runtime/ooc/primitives/OOCPrimitive.java 60.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2605      +/-   ##
============================================
- Coverage     71.39%   71.39%   -0.01%     
- Complexity    50539    50597      +58     
============================================
  Files          1632     1633       +1     
  Lines        196237   196588     +351     
  Branches      38201    38242      +41     
============================================
+ Hits         140113   140351     +238     
- Misses        45099    45186      +87     
- Partials      11025    11051      +26     

☔ 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 aa6857d into apache:main Sep 3, 2026
51 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in SystemDS PR Queue Sep 3, 2026
@janniklinde
janniklinde deleted the ooc/w1 branch October 1, 2026 12:49
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