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

workload/schemachanger: version gate JSON indexes in workload #103679

Merged
merged 1 commit into from
May 24, 2023

Conversation

fqazi
Copy link
Collaborator

@fqazi fqazi commented May 19, 2023

Previously, the randomized schema changer workload could try to use JSON indexes in a mixed version state where not all nodes supported them. This could lead to errors during the randomized workload. To address this, this patch version gates CREATE INDEX statements to avoid JSON indexes until the active version is high enough.

Fixes: #103452
Fixes #103758

Release note: None

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@fqazi fqazi force-pushed the fixJsonIndexColumns branch from 094584c to 276e688 Compare May 19, 2023 18:51
@fqazi fqazi changed the title worklaod/schemachanger: version gate JSON indexes in workload workload/schemachanger: version gate JSON indexes in workload May 19, 2023
@fqazi fqazi force-pushed the fixJsonIndexColumns branch from 276e688 to 6590583 Compare May 19, 2023 18:52
@fqazi fqazi marked this pull request as ready for review May 19, 2023 18:53
@fqazi fqazi requested a review from a team as a code owner May 19, 2023 18:53
@fqazi fqazi requested review from herkolategan, renatolabs and a team and removed request for a team May 19, 2023 18:53
@fqazi fqazi force-pushed the fixJsonIndexColumns branch 3 times, most recently from 0101790 to 52c839c Compare May 24, 2023 14:11
@fqazi fqazi requested a review from rafiss May 24, 2023 14:11
@fqazi fqazi force-pushed the fixJsonIndexColumns branch 3 times, most recently from 6614581 to 0ea367e Compare May 24, 2023 14:47
Previously, the randomized schema changer workload could try to
use JSON indexes in a mixed version state where not all nodes
supported them. This could lead to errors during the randomized
workload. To address this, this patch version gates CREATE INDEX
statements to avoid JSON indexes until the active version is high
enough.

Fixes: cockroachdb#103452
Release note: None
@fqazi fqazi force-pushed the fixJsonIndexColumns branch from 0ea367e to bfc1c5b Compare May 24, 2023 16:52
@fqazi
Copy link
Collaborator Author

fqazi commented May 24, 2023

@rafiss TFTR!

bors r+

@craig
Copy link
Contributor

craig bot commented May 24, 2023

This PR was included in a batch that timed out, it will be automatically retried

@craig
Copy link
Contributor

craig bot commented May 24, 2023

Build succeeded:

@craig craig bot merged commit aaaae95 into cockroachdb:master May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

roachtest: schemachange/mixed-versions failed roachtest: acceptance/mixed version flaked on master
3 participants