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

logictest: use high priority txn in partial_index and synthetic_privileges tests #133400

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

rafiss
Copy link
Collaborator

@rafiss rafiss commented Oct 24, 2024

This should make it less likely for the transaction to be aborted.

fixes #126763
fixes #132919
Release note: None

@rafiss rafiss added backport-24.1.x Flags PRs that need to be backported to 24.1. backport-24.2.x Flags PRs that need to be backported to 24.2 backport-24.3.x Flags PRs that need to be backported to 24.3 labels Oct 24, 2024
@rafiss rafiss requested review from Dedej-Bergin and a team October 24, 2024 20:22
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Contributor

@Dedej-Bergin Dedej-Bergin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained

…leges tests

This should make it less likely for the transaction to be aborted.

Release note: None
@rafiss rafiss force-pushed the deflake-partial-idx branch from 82075ec to 7a73029 Compare October 24, 2024 20:42
@rafiss rafiss changed the title logictest: use high priority txn in partial_index test logictest: use high priority txn in partial_index and synthetic_privileges tests Oct 24, 2024
@rafiss
Copy link
Collaborator Author

rafiss commented Oct 24, 2024

bors r+

@craig craig bot merged commit 05cd8a1 into cockroachdb:master Oct 24, 2024
22 of 23 checks passed
Copy link

blathers-crl bot commented Oct 24, 2024

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 #132919: branch-release-24.1, branch-release-24.2, branch-release-24.3.


Issue #126763: 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-24.1.x Flags PRs that need to be backported to 24.1. backport-24.2.x Flags PRs that need to be backported to 24.2 backport-24.3.x Flags PRs that need to be backported to 24.3
Projects
None yet
3 participants