Cache WorkerState host property

[fjetter]

See #9043

[Copilot]

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

distributed/scheduler.py:569

• Caching the host property at initialization may lead to inconsistencies if self.address changes later. Please confirm that the worker's address remains constant after initialization.

self.host = get_address_host(self.address)

distributed/scheduler.py:587

• Removing the dynamic host property in favor of a cached value requires ensuring that no dependent code relies on its dynamic computation. Verify that this change will not affect behaviors expecting an updated host value.

@property
    def host(self) -> str:

[github-actions]

Unit Test Results

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

    27 files  ±0      27 suites  ±0   11h 21m 52s ⏱️ + 5m 42s
 4 100 tests ±0   3 981 ✅  - 1    112 💤 ±0  7 ❌ +1 
51 400 runs  ±0  49 085 ✅ ±0  2 308 💤  - 1  7 ❌ +1 

For more details on these failures, see this check.

Results for commit 53fad01. ± Comparison against base commit 16aa189.

♻️ This comment has been updated with latest results.

[jrbourbeau]

LGTM -- test failures seem unrelated