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

Add a check in debug doctor for a cluster at risk of indexes with invalid encoding #106794

Closed
smcvey opened this issue Jul 13, 2023 · 1 comment
Closed
Assignees
Labels
C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) O-support Would prevent or help troubleshoot a customer escalation - bugs, missing observability/tooling, docs T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)

Comments

@smcvey
Copy link
Contributor

smcvey commented Jul 13, 2023

Is your feature request related to a problem? Please describe.
Please add an additional check to cockroach debug doctor which checks the descriptors looking for tables that are at risk of invalid encoding.

Additional context
We only need to have this check in the latest 23.1 binary, there's no need to backport it to earlier versions.

Jira issue: CRDB-29717

Epic CRDB-28665

@smcvey smcvey added the C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) label Jul 13, 2023
@rimadeodhar rimadeodhar changed the title Add a check in debug doctor for a cluster at risk of TA 99561 Add a check in debug doctor for a cluster at risk of indexes with invalid encoding Jul 13, 2023
@rafiss rafiss added the T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) label Jul 14, 2023
@rafiss rafiss added the O-support Would prevent or help troubleshoot a customer escalation - bugs, missing observability/tooling, docs label Jul 14, 2023
@rafiss
Copy link
Collaborator

rafiss commented Jul 14, 2023

Related to #106739 also.

fqazi added a commit to fqazi/cockroach that referenced this issue Jul 14, 2023
…olumns

Previously, we had a bug were it was possible that the primary indexes
was not storing some non-virtual columns. Unfortunately, adding
validation in all cases would break existing customers, so as
a safety we have added validation to crdb_internal.invalid_objects
and the debug doctor command. To address this, this patch adds a
new validation level only for invalid_objects / doctor.

Fixes: cockroachdb#106794
Informs: cockroachdb#106739

Release note: None
fqazi added a commit to fqazi/cockroach that referenced this issue Jul 18, 2023
…olumns

Previously, we had a bug were it was possible that the primary indexes
was not storing some non-virtual columns. Unfortunately, adding
validation in all cases would break existing customers, so as
a safety we have added validation to crdb_internal.invalid_objects
and the debug doctor command. To address this, this patch adds a
new validation level only for invalid_objects / doctor.

Fixes: cockroachdb#106794
Informs: cockroachdb#106739

Release note: None
@rafiss rafiss closed this as completed Jul 18, 2023
rafiss pushed a commit that referenced this issue Jul 18, 2023
…olumns

Previously, we had a bug were it was possible that the primary indexes
was not storing some non-virtual columns. Unfortunately, adding
validation in all cases would break existing customers, so as
a safety we have added validation to crdb_internal.invalid_objects
and the debug doctor command. To address this, this patch adds a
new validation level only for invalid_objects / doctor.

Fixes: #106794
Informs: #106739

Release note: None
rafiss pushed a commit that referenced this issue Jul 18, 2023
…olumns

Previously, we had a bug were it was possible that the primary indexes
was not storing some non-virtual columns. Unfortunately, adding
validation in all cases would break existing customers, so as
a safety we have added validation to crdb_internal.invalid_objects
and the debug doctor command. To address this, this patch adds a
new validation level only for invalid_objects / doctor.

Fixes: #106794
Informs: #106739

Release note: None
celiala pushed a commit that referenced this issue Jul 26, 2023
…olumns

Previously, we had a bug were it was possible that the primary indexes
was not storing some non-virtual columns. Unfortunately, adding
validation in all cases would break existing customers, so as
a safety we have added validation to crdb_internal.invalid_objects
and the debug doctor command. To address this, this patch adds a
new validation level only for invalid_objects / doctor.

Fixes: #106794
Informs: #106739

Release note: None

(cherry picked from commit 353e9c6)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) O-support Would prevent or help troubleshoot a customer escalation - bugs, missing observability/tooling, docs T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)
Projects
None yet
Development

No branches or pull requests

3 participants