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

logictestccl: flake in TestCCLLogic_multi_region_remote_access_error #102292

Closed
yuzefovich opened this issue Apr 25, 2023 · 1 comment · Fixed by #102484
Closed

logictestccl: flake in TestCCLLogic_multi_region_remote_access_error #102292

yuzefovich opened this issue Apr 25, 2023 · 1 comment · Fixed by #102484
Assignees
@msirek
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-sql-queries SQL Queries Team

Comments

@yuzefovich
Copy link
Member

yuzefovich commented Apr 25, 2023 •
edited by cockroach-jira-scripts
Loading

Seen here:

        /home/roach/.cache/bazel/_bazel_roach/c5a4e7d36696d9cd970af2045211a7df/sandbox/processwrapper-sandbox/1112/execroot/com_github_cockroachdb_cockroach/bazel-out/k8-fastbuild/bin/pkg/ccl/logictestccl/tests/multiregion-9node-3region-3azs-vec-off/multiregion-9node-3region-3azs-vec-off_test_/multiregion-9node-3region-3azs-vec-off_test.runfiles/com_github_cockroachdb_cockroach/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error:343: WITH vtab AS (SELECT * FROM messages_rbt)
        SELECT vtab.account_id, vtab.message FROM vtab
          INNER LOOKUP JOIN messages_rbr rbr on vtab.account_id = rbr.account_id and rbr.crdb_region = 'ap-southeast-2'
        expected success, but found
        (XCHR2) Query has no home region. Try adding a LIMIT clause. For more information, see https://www.cockroachlabs.com/docs/stable/cost-based-optimizer.html#control-whether-queries-are-limited-to-a-single-region
        relational.go:2118: in buildDistribute()

We're probably missing a retry?

Jira issue: CRDB-27384
@yuzefovich yuzefovich added the C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. label Apr 25, 2023
@blathers-crl blathers-crl bot added the T-sql-queries SQL Queries Team label Apr 25, 2023
@yuzefovich yuzefovich mentioned this issue Apr 25, 2023
Merged
@andrewbaptist
Copy link
Collaborator

Seen here also. This had #102223 merged

@rharding6373 rharding6373 mentioned this issue Apr 27, 2023
Merged
@craig craig bot closed this as completed in 79f42d2 Apr 27, 2023
@craig craig bot closed this as completed in #102484 Apr 27, 2023
@blathers-crl blathers-crl bot mentioned this issue Apr 27, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@msirek msirek

Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-sql-queries SQL Queries Team
Projects
SQL Queries
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

logictestccl: add missing retries to multi_region_remote_access_error test msirek/cockroach
3 participants
@msirek @yuzefovich @andrewbaptist