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

ttljob: stats query can cause job to run for much longer than it needs to #124305

Closed
rafiss opened this issue May 16, 2024 · 0 comments · Fixed by #124306
Closed

ttljob: stats query can cause job to run for much longer than it needs to #124305

rafiss opened this issue May 16, 2024 · 0 comments · Fixed by #124306
Assignees
@rafiss
Labels
A-row-level-ttl C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)
Projects
SQL Foundations

Comments

@rafiss
Copy link
Collaborator

rafiss commented May 16, 2024 •
edited by cockroach-jira-scripts

Describe the problem

In https://cockroachlabs.slack.com/archives/CHVV403F0/p1713541787550359 we had a report of a TTL job that stopped processing rows in ~5 hours, but needed 25 hours total for the job to run.

It seems that the issue is that row level TTL stats were on, and the query to load the stats was taking 25 hours.

Expected behavior

If the TTL job finishes, we should just cancel the stats query.

Epic CRDB-18322

Jira issue: CRDB-38813
@rafiss rafiss added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) A-row-level-ttl labels May 16, 2024
@rafiss rafiss self-assigned this May 16, 2024
@blathers-crl blathers-crl bot added this to Triage in SQL Foundations May 16, 2024
@rafiss rafiss mentioned this issue May 16, 2024
Merged
@craig craig bot closed this as completed in a19cc3f May 23, 2024
SQL Foundations automation moved this from Triage to Done May 23, 2024
This was referenced May 23, 2024
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rafiss rafiss

Labels
A-row-level-ttl C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)
Projects
SQL Foundations
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ttljob: don't block job completion on stats queries rafiss/cockroach
1 participant
@rafiss