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.

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

[custom-build] sql/rowexec: subject column backfills to admission control #79117

Closed
wants to merge 3 commits into from

Conversation

erikgrinaker
Copy link
Contributor

Temporary 21.2 backport of #79115 and friends, for a custom build.

@erikgrinaker erikgrinaker requested a review from a team as a code owner March 31, 2022 12:30
@erikgrinaker erikgrinaker self-assigned this Mar 31, 2022
@erikgrinaker erikgrinaker marked this pull request as draft March 31, 2022 12:30
@cockroach-teamcity
Copy link
Member

This change is Reviewable

dt and others added 2 commits April 4, 2022 12:03
Release note (bug fix): Previously IMPORT INTO could create duplicate entries UNIQUE constraints in REGIONAL BY ROW tables and tables utilizing UNIQUE WITHOUT INDEX constraints. A new post-IMPORT validation step for those tables now fails and rolls back the IMPORT in such cases.
Before this patch, when no bytes were admitted in an interval, we'd assume that
the correct estimate per byte was 1. This meant that we could see a huge jump
in tokens which effectively disabled the throttling of admission control.

Release note (bug fix): Fixed a bug in IO admission control which could result
in admission control failing to rate-limit when traffic was stalled such that
no work was admitted, despite the store's being in an unhealthy state.
@ajwerner ajwerner force-pushed the backfill-admission-control-21.2.7 branch 2 times, most recently from f8c1f60 to 31c5d9d Compare April 5, 2022 20:46
@ajwerner ajwerner force-pushed the backfill-admission-control-21.2.7 branch from 31c5d9d to 38835a5 Compare April 5, 2022 20:50
@erikgrinaker erikgrinaker deleted the backfill-admission-control-21.2.7 branch August 1, 2022 15:19
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.

4 participants