-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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: deflake TestBackfillWithProtectedTS #139774
sql: deflake TestBackfillWithProtectedTS #139774
Conversation
We have seen that the DELETE and INSERT step can fail in this test, which occurs before we manually send the table into the GC queue. To reduce the chance of this happening, we use a longer gc.ttl. Release note: None
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @rafiss)
bors r+ |
Based on the specified backports for this PR, I applied new labels to the following linked issue(s). Please adjust the labels as needed to match the branches actually affected by the issue(s), including adding any known older branches. Issue #139775: branch-release-24.2, branch-release-24.3, branch-release-25.1. Issue #139493: branch-release-24.1, branch-release-24.2, branch-release-24.3, branch-release-25.1. Issue #139281: branch-release-24.1, branch-release-24.2, branch-release-24.3. 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
We have seen that the DELETE and INSERT step can fail in this test, which occurs before we manually send the table into the GC queue. To reduce the chance of this happening, we use a longer gc.ttl.
fixes #139493
fixes #139281
fixes #139775
Release note: None