Skip to content

[unsupervised AI] Fix late worker alias restrictions - #9344

Open
AG0708 wants to merge 1 commit into
dask:mainfrom
AG0708:codex/8862-late-worker-alias-restrictions
Open

[unsupervised AI] Fix late worker alias restrictions#9344
AG0708 wants to merge 1 commit into
dask:mainfrom
AG0708:codex/8862-late-worker-alias-restrictions

Conversation

@AG0708

@AG0708 AG0708 commented Aug 11, 2026

Copy link
Copy Markdown

Summary

  • resolve worker-name aliases that connect after task submission to their exact worker address
  • prevent a late alias from incorrectly expanding the restriction to every worker on the same host
  • add a scheduler regression test covering mixed present and late worker names

Validation

  • distributed/tests/test_scheduler.py — 475 passed, 43 skipped, 1 expected xfail
  • focused late-worker regression — red on the base commit, green with this change
  • Ruff 0.15.16 lint and formatting checks — passed
  • git diff --check — passed

Closes #8862

Resolve a worker alias that connects after task submission to its exact address instead of expanding the restriction to every worker on the same host. Add a regression test for mixed present and late worker names.

Signed-off-by: Abhinav Gorrepati <gorrepatiabhinav1@gmail.com>
@github-actions

Copy link
Copy Markdown
Contributor

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

    40 files  +    1      40 suites  +1   14h 45m 25s ⏱️ + 17m 44s
 4 160 tests +    1   3 978 ✅  -     2    178 💤 ±  0  4 ❌ +3 
80 959 runs  +2 731  76 719 ✅ +2 588  4 236 💤 +140  4 ❌ +3 

For more details on these failures, see this check.

Results for commit f226a45. ± Comparison against base commit 40fcd99.

@AG0708
AG0708 marked this pull request as ready for review August 12, 2026 18:13
@AG0708
AG0708 requested a review from fjetter as a code owner August 12, 2026 18:13
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.

Client submit with workers doesn’t handle new joining workers correctly

1 participant