Skip to content

clarify dask priority docs#21581

Open
zzstoatzz wants to merge 1 commit intomainfrom
codex/clarify-dask-priority-docs
Open

clarify dask priority docs#21581
zzstoatzz wants to merge 1 commit intomainfrom
codex/clarify-dask-priority-docs

Conversation

@zzstoatzz
Copy link
Copy Markdown
Collaborator

Summary

This PR clarifies the prefect-dask docs around Dask task priorities.

Why

The current example can be read as implying that a later high-priority .submit() call will reorder execution ahead of earlier low-priority submissions. In practice, Dask priority is a soft scheduling hint for queued work and does not preempt tasks that have already been assigned to a worker.

Impact

Readers should have a more accurate expectation of how dask.annotate(priority=...) behaves when used with DaskTaskRunner.

Root cause

The behavior discussed in #21568 is not caused by Prefect overriding Dask annotations. A local repro against a real Prefect OSS server showed that the annotations reach Dask, while execution may still begin in submission order for already-assigned work.

Validation

  • curl http://localhost:4200/api/health
  • uv run prefect config view
  • uv run --extra dask python repros/21568.py

@mintlify
Copy link
Copy Markdown

mintlify bot commented Apr 16, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Prefect 🟢 Ready View Preview Apr 16, 2026, 7:03 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@github-actions github-actions bot added the docs label Apr 16, 2026
@zzstoatzz zzstoatzz changed the title [codex] clarify dask priority docs clarify dask priority docs Apr 17, 2026
@zzstoatzz zzstoatzz marked this pull request as ready for review April 17, 2026 15:58
@zzstoatzz zzstoatzz added the integrations Related to integrations with other services label Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs integrations Related to integrations with other services

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants