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

release-23.1: workload/schemachange: handle desc ID generator errors #126905

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

fqazi
Copy link
Collaborator

@fqazi fqazi commented Jul 9, 2024

Previously, during migrations to version 23.1, if a descriptor ID generator error occurred during CREATE SCHEMA, the process would incorrectly fail in this test. This error is expected due to the addition of a new sequence for tracking descriptor IDs in version 23.1. To resolve this, this patch introduces logic to detect and ignore this error specifically for CREATE SCHEMA operations.

Fixes: #126784
Release justification: test only change
Release note: None

Previously, during migrations to version 23.1, if a descriptor ID
generator error occurred during CREATE SCHEMA, the process would
incorrectly fail in this test. This error is expected due to the
addition of a new sequence for tracking descriptor IDs in version 23.1.
To resolve this, this patch introduces logic to detect and ignore this
error specifically for CREATE SCHEMA operations.

Fixes: cockroachdb#126784

Release note: None
@fqazi fqazi requested a review from a team July 9, 2024 19:22
@fqazi fqazi requested a review from a team as a code owner July 9, 2024 19:22
@fqazi fqazi requested review from herkolategan and renatolabs and removed request for a team July 9, 2024 19:22
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@fqazi fqazi changed the title girelease-23.1: workload/schemachange: handle desc ID generator errors release-23.1: workload/schemachange: handle desc ID generator errors Jul 9, 2024
Copy link
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.

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @herkolategan and @renatolabs)

@fqazi fqazi merged commit 0c096d6 into cockroachdb:release-23.1 Jul 10, 2024
6 checks passed
@fqazi
Copy link
Collaborator Author

fqazi commented Jul 11, 2024

blathers backport release-23.1.24-rc

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.

3 participants