Skip to content

[improvement](fe) Make max_user_connections upper bound follow qe_max_connection - #67622

Open
HonestManXin wants to merge 1 commit into
apache:masterfrom
HonestManXin:connection
Open

[improvement](fe) Make max_user_connections upper bound follow qe_max_connection#67622
HonestManXin wants to merge 1 commit into
apache:masterfrom
HonestManXin:connection

Conversation

@HonestManXin

Copy link
Copy Markdown
Contributor

Previously the per-user connection limit max_user_connections was hard-coded to a maximum of 10000. When operators raised the FE global connection limit qe_max_connection above 10000, a single user could not be granted more than 10000 connections, which is unreasonable since the effective limit is already bounded by the global setting.

Now the upper bound of max_user_connections is Math.max(qe_max_connection,10000), so a user can be configured up to the global limit while still keeping the original 10000 floor for backward compatibility.

@hello-stephen

Copy link
Copy Markdown
Contributor

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@HonestManXin HonestManXin changed the title [improvement](fe) Make max_user_connections upper bound follow qe_max… [improvement](fe) Make max_user_connections upper bound follow qe_max_connection Sep 7, 2026
@HonestManXin

Copy link
Copy Markdown
Contributor Author

/review

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Codex automated review failed and did not complete.

Error: Selected model is at capacity. Please try a different model.
Workflow run: https://github.com/apache/doris/actions/runs/34124264720

Please inspect the workflow logs and rerun the review after the underlying issue is resolved.

@HonestManXin

Copy link
Copy Markdown
Contributor Author

run buildall

@hello-stephen

Copy link
Copy Markdown
Contributor

FE UT Coverage Report

Increment line coverage 33.33% (1/3) 🎉
Increment coverage report
Complete coverage report

@hello-stephen

Copy link
Copy Markdown
Contributor

FE Regression Coverage Report

Increment line coverage 2.04% (1/49) 🎉
Increment coverage report
Complete coverage report

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.

2 participants