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

sql: Observability improvements meta issue for TTL deletions jobs #81905

Closed
1 task
vy-ton opened this issue May 26, 2022 · 4 comments · Fixed by #86801
Closed
1 task

sql: Observability improvements meta issue for TTL deletions jobs #81905

vy-ton opened this issue May 26, 2022 · 4 comments · Fixed by #86801
Assignees
@rafiss
Labels
A-row-level-ttl C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)

Comments

@vy-ton
Copy link
Contributor

vy-ton commented May 26, 2022 •
edited by exalate-issue-sync bot
Loading

Jira issue: CRDB-16123
Epic CRDB-14592
@vy-ton vy-ton added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) A-row-level-ttl labels May 26, 2022
@exalate-issue-sync exalate-issue-sync bot added the T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) label May 26, 2022
@rafiss rafiss self-assigned this Aug 24, 2022
@rafiss rafiss mentioned this issue Aug 24, 2022
Merged
@rafiss
Copy link
Collaborator

rafiss commented Aug 24, 2022

@vy-ton we can't include the exact queries in the job description, since the job runs many different queries as it iterates through the rows to be deleted. but, we can show the query templates with something like this

image

@rafiss
Copy link
Collaborator

rafiss commented Aug 24, 2022

@vy could you look at the description of #86801 and confirm if that is an acceptable solution.

@vy-ton
Copy link
Contributor Author

vy-ton commented Aug 25, 2022

@rafiss this looks great! Could we make the <x> correspond to the TTL setting names?

@rafiss
Copy link
Collaborator

rafiss commented Aug 25, 2022

sure, good idea

@craig craig bot closed this as completed in 3688055 Aug 26, 2022
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-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ttl: add queries to job details rafiss/cockroach
3 participants
@rafiss @ecwall @vy-ton