Skip to content
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.

Sign up for GitHub

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

Jump to bottom

core/sql: admission control tracking issue #38076

Closed
awoods187 opened this issue Jun 6, 2019 · 3 comments
Closed

core/sql: admission control tracking issue #38076

awoods187 opened this issue Jun 6, 2019 · 3 comments
Assignees
@RaduBerinde
Labels
A-admission-control A-kv Anything in KV that doesn't belong in a more specific category. A-kv-client Relating to the KV client and the KV interface. C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-kv KV Team T-sql-queries SQL Queries Team

Comments

@awoods187
Copy link
Contributor

awoods187 commented Jun 6, 2019 •
edited by RoachietheSupportRoach
Loading

Admission control limits the rate of query execution across the cluster to levels that the system can handle without significant performance degradation. Once resource limits have been reached, and the cluster is overloaded (e.g. RAM, CPU, I/O, Network), incoming queries must wait to begin execution. These queries are stalled (queued) and are admitted (that is, begin executing) when the resources become available. If the time queries spend in the queue + execution exceed a connection time limit the application requests will no longer be stalled and are outright rejected. DBAs can set when admission control will begin queuing, a maximum number of connections per node, and a query timeout.

This is a tracking issue to support this work.

gz#6246
@awoods187 awoods187 added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) A-kv Anything in KV that doesn't belong in a more specific category. labels Jun 6, 2019
@lunevalex lunevalex added the A-kv-client Relating to the KV client and the KV interface. label Jul 28, 2020
@BramGruneir
Copy link
Member

@awoods187 what's the status of this issue?

@awoods187
Copy link
Contributor Author

We are investigating this issue but have not yet begun development work.

@awoods187 awoods187 assigned RaduBerinde and unassigned ajwerner Apr 28, 2021
@jlinder jlinder added T-sql-queries SQL Queries Team T-kv KV Team labels Jun 16, 2021
@lunevalex
Copy link
Collaborator

With #68535 merged, I am going to close this issues as complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RaduBerinde RaduBerinde

Labels
A-admission-control A-kv Anything in KV that doesn't belong in a more specific category. A-kv-client Relating to the KV client and the KV interface. C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-kv KV Team T-sql-queries SQL Queries Team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@jlinder @BramGruneir @ajwerner @RaduBerinde @awoods187 @lunevalex