Skip to content

[OOC] Adapt OOCStream to Forward QueueCallbacks - #2537

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

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

Conversation

@janniklinde

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 40.42553% with 56 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.59%. Comparing base (108229b) to head (976c61d).

Files with missing lines Patch % Lines
...untime/instructions/ooc/SubscribableTaskQueue.java 66.00% 11 Missing and 6 partials ⚠️
...sysds/runtime/instructions/ooc/PlaybackStream.java 5.88% 12 Missing and 4 partials ⚠️
...he/sysds/runtime/ooc/stream/FilteredOOCStream.java 0.00% 12 Missing ⚠️
...ache/sysds/runtime/ooc/stream/MergedOOCStream.java 28.57% 5 Missing ⚠️
.../apache/sysds/runtime/ooc/stream/SubOOCStream.java 33.33% 4 Missing ⚠️
...e/sysds/runtime/ooc/stream/SplittingOOCStream.java 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2537      +/-   ##
============================================
- Coverage     71.61%   71.59%   -0.03%     
+ Complexity    49873    49849      -24     
============================================
  Files          1602     1602              
  Lines        193007   193054      +47     
  Branches      37787    37792       +5     
============================================
- Hits         138226   138208      -18     
- Misses        44015    44066      +51     
- Partials      10766    10780      +14     

☔ 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 e557836 into apache:main Jul 7, 2026
51 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
@janniklinde
janniklinde deleted the ooc/planner-merge-6 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