Skip to content

Fix JET errors regarding the global cluster_manager being defined/undefined#176

Closed
DilumAluthge wants to merge 1 commit into
masterfrom
dpa/global-cluster_manager-is-defined
Closed

Fix JET errors regarding the global cluster_manager being defined/undefined#176
DilumAluthge wants to merge 1 commit into
masterfrom
dpa/global-cluster_manager-is-defined

Conversation

@DilumAluthge

Copy link
Copy Markdown
Member

This PR fixes the JET error, but I'm not sure if it's the right approach.


┌  @ Distributed /workpath/Distributed.jl/src/process_messages.jl:319
│ `Distributed.cluster_manager` may be undefined: Distributed.cluster_manager
└────────────────────
┌  @ Distributed /workpath/Distributed.jl/src/process_messages.jl:332
│ `Distributed.cluster_manager` may be undefined: Distributed.cluster_manager
└────────────────────

We know that the global cluster_manager is defined, because it was defined in the init_worker(...) function.

(cherry picked from commit 94de3a2)

@codecov

codecov Bot commented Feb 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.36%. Comparing base (871e3d7) to head (1dbe448).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #176      +/-   ##
==========================================
+ Coverage   79.34%   79.36%   +0.02%     
==========================================
  Files          10       10              
  Lines        1951     1953       +2     
==========================================
+ Hits         1548     1550       +2     
  Misses        403      403              

☔ 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.

…ndefined

```
┌  @ Distributed /workpath/Distributed.jl/src/process_messages.jl:319
│ `Distributed.cluster_manager` may be undefined: Distributed.cluster_manager
└────────────────────
┌  @ Distributed /workpath/Distributed.jl/src/process_messages.jl:332
│ `Distributed.cluster_manager` may be undefined: Distributed.cluster_manager
└────────────────────
```

We know that the global `cluster_manager` is defined, because it was defined in
the `init_worker(...)` function.

This commit fixes the JET error.

(cherry picked from commit 94de3a2)
@DilumAluthge
DilumAluthge force-pushed the dpa/global-cluster_manager-is-defined branch from 8f0871d to 1dbe448 Compare February 13, 2026 00:03
@DilumAluthge
DilumAluthge deleted the dpa/global-cluster_manager-is-defined branch February 20, 2026 13:19
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.

1 participant