Skip to content

logictest: filter cluster_locks queries in select_for_share to test database#168751

Merged
trunk-io[bot] merged 1 commit intocockroachdb:masterfrom
stevendanna:ssd/fix-168339
May 6, 2026
Merged

logictest: filter cluster_locks queries in select_for_share to test database#168751
trunk-io[bot] merged 1 commit intocockroachdb:masterfrom
stevendanna:ssd/fix-168339

Conversation

@stevendanna
Copy link
Copy Markdown
Collaborator

@stevendanna stevendanna commented Apr 21, 2026

With virtual intent resolution (VIR) enabled metamorphically, bootstrap
intents on system.role_members can persist in the lock table and appear
in unfiltered crdb_internal.cluster_locks queries, causing the test to
see unexpected rows.

Fixes #168339
Epic: none
Release note: None

…atabase

With virtual intent resolution (VIR) enabled metamorphically, bootstrap
intents on system.role_members can persist in the lock table and appear
in unfiltered crdb_internal.cluster_locks queries, causing the test to
see unexpected rows.

Fixes cockroachdb#168339
Release note: None

Co-Authored-By: roachdev-claude <roachdev-claude-bot@cockroachlabs.com>
@trunk-io
Copy link
Copy Markdown
Contributor

trunk-io Bot commented Apr 21, 2026

😎 Merged successfully - details.

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@stevendanna stevendanna requested a review from rafiss April 29, 2026 07:51
Copy link
Copy Markdown
Collaborator

@rafiss rafiss left a comment

Choose a reason for hiding this comment

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

thanks for the fix! I'll go ahead and merge since this deflakes the test

/trunk merge

@blathers-crl
Copy link
Copy Markdown

blathers-crl Bot commented May 6, 2026

Detected infrastructure failure on trunk-merge branch (matched: self-hosted runner lost communication with the server). Automatically resubmitting to merge queue (attempt 1 of 2). (run link)

@blathers-crl
Copy link
Copy Markdown

blathers-crl Bot commented May 6, 2026

/trunk merge

@trunk-io trunk-io Bot merged commit 88ba9be into cockroachdb:master May 6, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pkg/sql/logictest/tests/fakedist-vec-off/fakedist-vec-off_test: TestLogic_select_for_share failed

3 participants