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

sql: sql stmt statistics jobs have massive negative impact on oltp_read_only #135988

Open
tbg opened this issue Nov 22, 2024 · 4 comments
Open

sql: sql stmt statistics jobs have massive negative impact on oltp_read_only #135988

tbg opened this issue Nov 22, 2024 · 4 comments
Labels
branch-master Failures and bugs on the master branch. C-performance Perf of queries or internals. Solution not expected to change functional behavior. o-perf-efficiency Related to performance efficiency P-1 Issues/test failures with a fix SLA of 1 month T-observability

Comments

@tbg
Copy link
Member

tbg commented Nov 22, 2024

See here.

image

Epic: CRDB-42584

Jira issue: CRDB-44818

@tbg tbg added C-performance Perf of queries or internals. Solution not expected to change functional behavior. branch-master Failures and bugs on the master branch. T-sql-queries SQL Queries Team P-1 Issues/test failures with a fix SLA of 1 month o-perf-efficiency Related to performance efficiency labels Nov 22, 2024
Copy link

blathers-crl bot commented Nov 22, 2024

This issue has multiple T-eam labels. Please make sure it only has one, or else issue synchronization will not work correctly.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@michae2
Copy link
Collaborator

michae2 commented Nov 26, 2024

@tbg could you clarify whether this is about SQL table statistics, or observability statement statistics? They're both informally called "SQL stats" unfortunately, but they are not the same.

@michae2
Copy link
Collaborator

michae2 commented Nov 26, 2024

Oh, based on this thread, it seems to be the SQL statement stats owned by observability? Moving.

@michae2 michae2 added T-observability and removed T-sql-queries SQL Queries Team labels Nov 26, 2024
@michae2 michae2 removed this from SQL Queries Nov 26, 2024
@ajstorm
Copy link
Collaborator

ajstorm commented Nov 27, 2024

@tbg I opened #136324 for the specific issue of flushing SQL stats. Along with #135996, does that cover everything in this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch-master Failures and bugs on the master branch. C-performance Perf of queries or internals. Solution not expected to change functional behavior. o-perf-efficiency Related to performance efficiency P-1 Issues/test failures with a fix SLA of 1 month T-observability
Projects
None yet
Development

No branches or pull requests

3 participants