Skip to content

Fix a JET error around matching methods for getindex(...)#170

Merged
DilumAluthge merged 1 commit into
masterfrom
dpa/cherrypick-jet-getindex
Feb 21, 2026
Merged

Fix a JET error around matching methods for getindex(...)#170
DilumAluthge merged 1 commit into
masterfrom
dpa/cherrypick-jet-getindex

Conversation

@DilumAluthge

Copy link
Copy Markdown
Member
┌ setup_launched_worker(manager::Distributed.SSHManager, wconfig::Distributed.WorkerConfig, launched_q::Vector{Int64}) @ Distributed /workpath/Distributed.jl/src/cluster.jl:563
│ no matching method found `getindex(::Nothing, ::Symbol)` (1/2 union split): cnt = ((wconfig::Distributed.WorkerConfig).environ::Union{Nothing, Dict})[:cpu_threads]
└────────────────────

(cherry picked from commit 61ff327)

```
┌ setup_launched_worker(manager::Distributed.SSHManager, wconfig::Distributed.WorkerConfig, launched_q::Vector{Int64}) @ Distributed /workpath/Distributed.jl/src/cluster.jl:563
│ no matching method found `getindex(::Nothing, ::Symbol)` (1/2 union split): cnt = ((wconfig::Distributed.WorkerConfig).environ::Union{Nothing, Dict})[:cpu_threads]
└────────────────────
```

(cherry picked from commit 61ff327)
@codecov

codecov Bot commented Feb 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 79.34%. Comparing base (871e3d7) to head (fec5037).
⚠️ Report is 8 commits behind head on master.

Files with missing lines Patch % Lines
src/cluster.jl 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #170   +/-   ##
=======================================
  Coverage   79.34%   79.34%           
=======================================
  Files          10       10           
  Lines        1951     1951           
=======================================
  Hits         1548     1548           
  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.

@JamesWrigley JamesWrigley left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@DilumAluthge DilumAluthge merged commit d06aa73 into master Feb 21, 2026
7 of 8 checks passed
@DilumAluthge DilumAluthge deleted the dpa/cherrypick-jet-getindex branch February 21, 2026 04:38
DilumAluthge added a commit to JuliaParallel/DistributedNext.jl that referenced this pull request Feb 22, 2026
This is a forward-port of JuliaLang/Distributed.jl#170 (JuliaLang/Distributed.jl@d06aa73).

```
┌ setup_launched_worker(manager::Distributed.SSHManager, wconfig::Distributed.WorkerConfig, launched_q::Vector{Int64}) @ Distributed /workpath/Distributed.jl/src/cluster.jl:563
│ no matching method found `getindex(::Nothing, ::Symbol)` (1/2 union split): cnt = ((wconfig::Distributed.WorkerConfig).environ::Union{Nothing, Dict})[:cpu_threads]
└────────────────────
```

(cherry picked from commit d06aa735f9511ac2618884e77e7caf29fdaeb1b1)
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.

2 participants