Skip to content

Clarify ResourceManager's allow-one guarantee - #30400

Closed
tamird wants to merge 1 commit into
bazelbuild:masterfrom
tamird:tamird/fix-resource-allow-one-docs
Closed

Clarify ResourceManager's allow-one guarantee#30400
tamird wants to merge 1 commit into
bazelbuild:masterfrom
tamird:tamird/fix-resource-allow-one-docs

Conversation

@tamird

@tamird tamird commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

The class documentation promises global progress, but the allow-one
policy only permits an unused resource type to exceed its capacity.
Worker quota and concurrency limits can still block acquisition. Narrow
the guarantee so callers do not rely on progress the manager cannot
provide.

The class documentation promises global progress, but the allow-one
policy only permits an unused resource type to exceed its capacity.
Worker quota and concurrency limits can still block acquisition. Narrow
the guarantee so callers do not rely on progress the manager cannot
provide.
@github-actions github-actions Bot added team-Performance Issues for Performance teams awaiting-review PR is awaiting review from an assigned reviewer labels Jul 21, 2026
@tamird

tamird commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

Extracted from #30310. Review of the scheduling change exposed that the
existing class documentation promises global progress even though
worker quota and concurrency limits can still block acquisition.

@meisterT
meisterT requested a review from bigelephant29 July 22, 2026 10:37
@bigelephant29 bigelephant29 added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Jul 27, 2026
@github-actions github-actions Bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Jul 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Performance Issues for Performance teams

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants