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

sqlsmith: expected 4 destination arguments in Scan, not 2 #78138

Closed
cockroach-teamcity opened this issue Mar 19, 2022 · 5 comments · Fixed by #84650
Closed

sqlsmith: expected 4 destination arguments in Scan, not 2 #78138

cockroach-teamcity opened this issue Mar 19, 2022 · 5 comments · Fixed by #84650
Assignees
@RichardJCai
Labels
branch-release-22.1 Used to mark GA and release blockers, technical advisories, and bugs for 22.1 C-test-failure Broken test (automatically or manually discovered). O-roachtest O-robot Originated from a bot. T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) T-sql-queries SQL Queries Team

Comments

@cockroach-teamcity
Copy link
Member

cockroach-teamcity commented Mar 19, 2022 •
edited by cockroach-jira-scripts
Loading

roachtest.sqlsmith/setup=seed-multi-region/setting=multi-region failed with artifacts on release-22.1 @ f5b42a4e3bbdadb25aee89a41871239a624d538d:

The test failed on branch=release-22.1, cloud=gce:
test artifacts and logs in: /artifacts/sqlsmith/setup=seed-multi-region/setting=multi-region/run_1
	sqlsmith.go:271,sqlsmith.go:304,test_runner.go:866: Failed generating a query Caught error sql: expected 4 destination arguments in Scan, not 2
Help

See: roachtest README

See: How To Investigate (internal)

/cc @cockroachdb/sql-queries

This test on roachdash | Improve this report!

Jira issue: CRDB-13987
@cockroach-teamcity cockroach-teamcity added branch-release-22.1 Used to mark GA and release blockers, technical advisories, and bugs for 22.1 C-test-failure Broken test (automatically or manually discovered). O-roachtest O-robot Originated from a bot. release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. labels Mar 19, 2022
@blathers-crl blathers-crl bot added the T-sql-queries SQL Queries Team label Mar 19, 2022
@rytaft
Copy link
Collaborator

rytaft commented Mar 21, 2022

Not a release blocker -- error happened when generating the query

@rytaft rytaft removed the release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. label Mar 21, 2022
@rytaft
Copy link
Collaborator

rytaft commented Mar 22, 2022

Looks like this is a problem with the multi-region SQLSmith setup. cc @RichardJCai

@rytaft rytaft changed the title roachtest: sqlsmith/setup=seed-multi-region/setting=multi-region failed sqlsmith: expected 4 destination arguments in Scan, not 2 Mar 22, 2022
@blathers-crl blathers-crl bot added the T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) label Mar 22, 2022
This was referenced May 2, 2022
Closed
Closed
This was referenced May 31, 2022
Closed
Closed
This was referenced Jun 12, 2022
Closed
Closed
@yuzefovich
Copy link
Member

@RichardJCai friendly ping on this - it has been happening fairly often.

@RichardJCai
Copy link
Contributor

@RichardJCai friendly ping on this - it has been happening fairly often.

I'll try to address it this week. At the very least might disable the test.

@RichardJCai
Copy link
Contributor

Oh yeah feel free to Slack me about these, sometimes these Github issues slide through my email

This was referenced Jul 5, 2022
Closed
Closed
@yuzefovich yuzefovich mentioned this issue Jul 9, 2022
Closed
@RichardJCai RichardJCai mentioned this issue Jul 19, 2022
Merged
@craig craig bot closed this as completed in b77d49e Jul 20, 2022
@blathers-crl blathers-crl bot mentioned this issue Jul 20, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RichardJCai RichardJCai

Labels
branch-release-22.1 Used to mark GA and release blockers, technical advisories, and bugs for 22.1 C-test-failure Broken test (automatically or manually discovered). O-roachtest O-robot Originated from a bot. T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) T-sql-queries SQL Queries Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

sql: fix multiregion sqlsmith failure RichardJCai/cockroach
4 participants
@rytaft @yuzefovich @cockroach-teamcity @RichardJCai