Skip to content

Clarify app-max-concurrency default value description - #5349

Open
minKasent wants to merge 1 commit into
dapr:v1.18from
minKasent:fix-concurrency-desc
Open

minKasent wants to merge 1 commit into
dapr:v1.18from
minKasent:fix-concurrency-desc

Conversation

@minKasent

Copy link
Copy Markdown

Thank you for helping make the Dapr documentation better!

Please follow this checklist before submitting:

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Read the contribution guide
  • Commands include options for Linux, MacOS, and Windows within tabpane
  • New file and folder names are globally unique
  • Page references use shortcodes instead of markdown or URL links
  • Images use HTML style and have alternative text
  • Places where multiple code/command options are given have tabpane

In addition, please fill out the following to help reviewers understand this pull request:

Description

In �rguments-annotations-overview.md, the --app-max-concurrency entry previously read Default value: -1, meaning no concurrency., which could be misread as imposing a single-request restriction. This PR updates the description to Default value: -1, meaning no concurrency limit is enforced., matching control-concurrency.md.

Issue reference

Closes #4606

Update default value explanation to clarify that -1 means no concurrency limit is enforced, aligning with control-concurrency.md.

Closes dapr#4606

Signed-off-by: Dang Khoa <dangkhoa3971@gmail.com>
@minKasent
minKasent requested review from a team as code owners October 2, 2026 07:55
Copilot AI balanced review requested due to automatic review settings October 2, 2026 07:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@CasperGN CasperGN assigned CasperGN and unassigned CasperGN Oct 12, 2026
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.

Typo in the documentation

3 participants