-
Notifications
You must be signed in to change notification settings - Fork 3.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable query request queuing by default when total laning is turned on. #15440
Enable query request queuing by default when total laning is turned on. #15440
Conversation
Perhaps we should point it out in the developer release notes in the next release. |
The docs for |
Updated the docs. |
Is this doc incorrect https://github.com/apache/druid/blob/master/docs/configuration/index.md?plain=1#L1777 |
No. It's correct. This PR queues requests when the total lane is at capacity but any other lane will still see the excess request being rejected. |
This PR enables the flag by default to queue excess query requests in the jetty queue. Still keeping the flag so that it can be turned off if necessary. But the flag will be removed in the future.
Release notes
MIN(Integer.MAX_VALUE, druid.server.http.maxQueryTimeout
)if value of
druid.server.http.maxQueryTimeout
is negative, then the excess requests will be queued forever.This PR has: