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

sentry: type_desc.go:689: type × (111): referencing table 113 was dropped without dependency unlinking (1) keys: [sql.schema.validation_errors.write.cross_references.type sql.schema.validation_errors.write.cr... #92456

Closed
cockroach-teamcity opened this issue Nov 24, 2022 · 1 comment
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. O-sentry Originated from an in-the-wild panic report.

Comments

@cockroach-teamcity
Copy link
Member

cockroach-teamcity commented Nov 24, 2022

This issue was autofiled by Sentry. It represents a crash or reported error on a live cluster with telemetry enabled.

Sentry link: https://sentry.io/organizations/cockroach-labs/issues/3764715944/?referrer=webhooks_plugin

Panic message:

type_desc.go:689: type × (111): referencing table 113 was dropped without dependency unlinking
(1) keys: [sql.schema.validation_errors.write.cross_references.type sql.schema.validation_errors.write.cross_references.type]
Wraps: (2) secondary error attachment
| type × (112): referencing table 113 was dropped without dependency unlinking
| (1) keys: [sql.schema.validation_errors.write.cross_references.type]
| Wraps: (2) attached stack trace
| -- stack trace:
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).decorate
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate/validate.go:220
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).Report
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate/validate.go:149
| | [...repeated from below...]
| Wraps: (3) type × (112)
| Wraps: (4) assertion failure
| Wraps: (5) attached stack trace
| -- stack trace:
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/typedesc.(*immutable).ValidateCrossReferences
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/typedesc/type_desc.go:689
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate.Validate.func2
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate/validate.go:77
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).validateDescriptorsAtLevel
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate/validate.go:165
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate.Validate
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate/validate.go:72
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/descs.(*Collection).Validate
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/descs/validate.go:38
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/descs.(*Collection).ValidateUncommittedDescriptors
| | github.com/cockroachdb/cockroach/pkg/sql/catalog/descs/validate.go:61
| | github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).commitSQLTransactionInternal
| | github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:1014
| | github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).commitSQLTransaction
| | github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:950
| | github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).execStmtInOpenState
| | github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:582
| | github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).execStmt
| | github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:179
| | github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).execCmd.func1
| | github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:1891
| | github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).execCmd
| | github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:1895
| | github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).run
| | github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:1817
| | github.com/cockroachdb/cockroach/pkg/sql.(*Server).ServeConn
| | github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:763
| | github.com/cockroachdb/cockroach/pkg/sql/pgwire.(*conn).processCommandsAsync.func1
| | github.com/cockroachdb/cockroach/pkg/sql/pgwire/conn.go:728
| | runtime.goexit
| | GOROOT/src/runtime/asm_amd64.s:1581
| Wraps: (6) referencing table 113 was dropped without dependency unlinking
| Error types: (1) *telemetrykeys.withTelemetry (2) *withstack.withStack (3) *errutil.withPrefix (4) *assert.withAssertionFailure (5) *withstack.withStack (6) *errutil.leafError
Wraps: (3) keys: [sql.schema.validation_errors.write.cross_references.type]
Wraps: (4) attached stack trace
-- stack trace:
| github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).decorate
| github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate/validate.go:220
| github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).Report
| github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate/validate.go:149
| [...repeated from below...]
Wraps: (5) type × (111)
Wraps: (6) assertion failure
Wraps: (7) attached stack trace
-- stack trace:
| github.com/cockroachdb/cockroach/pkg/sql/catalog/typedesc.(*immutable).ValidateCrossReferences
| github.com/cockroachdb/cockroach/pkg/sql/catalog/typedesc/type_desc.go:689
| github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate.Validate.func2
| github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate/validate.go:77
| github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).validateDescriptorsAtLevel
| github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate/validate.go:165
| github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate.Validate
| github.com/cockroachdb/cockroach/pkg/sql/catalog/internal/validate/validate.go:72
| github.com/cockroachdb/cockroach/pkg/sql/catalog/descs.(*Collection).Validate
| github.com/cockroachdb/cockroach/pkg/sql/catalog/descs/validate.go:38
| github.com/cockroachdb/cockroach/pkg/sql/catalog/descs.(*Collection).ValidateUncommittedDescriptors
| github.com/cockroachdb/cockroach/pkg/sql/catalog/descs/validate.go:61
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).commitSQLTransactionInternal
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:1014
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).commitSQLTransaction
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:950
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).execStmtInOpenState
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:582
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).execStmt
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:179
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).execCmd.func1
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:1891
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).execCmd
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:1895
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).run
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:1817
| github.com/cockroachdb/cockroach/pkg/sql.(*Server).ServeConn
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:763
| github.com/cockroachdb/cockroach/pkg/sql/pgwire.(*conn).processCommandsAsync.func1
| github.com/cockroachdb/cockroach/pkg/sql/pgwire/conn.go:728
| runtime.goexit
| GOROOT/src/runtime/asm_amd64.s:1581
Wraps: (8) referencing table 113 was dropped without dependency unlinking
Error types: (1) *telemetrykeys.withTelemetry (2) *secondary.withSecondaryError (3) *telemetrykeys.withTelemetry (4) *withstack.withStack (5) *errutil.withPrefix (6) *assert.withAssertionFailure (7) *withstack.withStack (8) *errutil.leafError
-- report composition:
*errutil.leafError: referencing table 113 was dropped without dependency unlinking
type_desc.go:689: *withstack.withStack (top exception)
*assert.withAssertionFailure
*errutil.withPrefix: type × (111)
validate.go:220: *withstack.withStack (1)
*telemetrykeys.withTelemetry: sql.schema.validation_errors.write.cross_references.type
*secondary.withSecondaryError: details for github.com/cockroachdb/errors/telemetrykeys/*telemetrykeys.withTelemetry:::
*telemetrykeys.withTelemetry: sql.schema.validation_errors.write.cross_references.type
(check the extra data payloads)

Stacktrace (expand for inline code snippets):

case catalog.Type:
err = errors.Wrapf(err, catalog.Type+" %q (%d)", name, id)
default:
in pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).decorate
}
vea.errors = append(vea.errors, vea.decorate(err))
}
in pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).Report
if tableDesc.Dropped() {
vea.Report(errors.AssertionFailedf(
"referencing table %d was dropped without dependency unlinking", id))
in pkg/sql/catalog/typedesc.(*immutable).ValidateCrossReferences
if !desc.Dropped() {
desc.ValidateCrossReferences(&vea, vdg)
}
in pkg/sql/catalog/internal/validate.Validate.func2
vea.currentDescriptor = desc
validationFn(desc)
}
in pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).validateDescriptorsAtLevel
// Descriptor cross-reference checks.
if !vea.validateDescriptorsAtLevel(
catalog.ValidationLevelCrossReferences,
in pkg/sql/catalog/internal/validate.Validate
version := tc.settings.Version.ActiveVersion(ctx)
return validate.Validate(
ctx,
in pkg/sql/catalog/descs.(*Collection).Validate
}
return tc.Validate(ctx, txn, catalog.ValidationWriteTelemetry, catalog.ValidationLevelAllPreTxnCommit, descs...)
}
in pkg/sql/catalog/descs.(*Collection).ValidateUncommittedDescriptors
if err := ex.extraTxnState.descCollection.ValidateUncommittedDescriptors(ctx, ex.state.mu.txn); err != nil {
return err
in pkg/sql.(*connExecutor).commitSQLTransactionInternal
ex.phaseTimes.SetSessionPhaseTime(sessionphase.SessionStartTransactionCommit, timeutil.Now())
if err := commitFn(ctx); err != nil {
return ex.makeErrEvent(err, ast)
in pkg/sql.(*connExecutor).commitSQLTransaction
// CommitTransaction is executed fully here; there's no plan for it.
ev, payload := ex.commitSQLTransaction(ctx, ast, ex.commitSQLTransactionInternal)
return ev, payload, nil
in pkg/sql.(*connExecutor).execStmtInOpenState
} else {
ev, payload, err = ex.execStmtInOpenState(ctx, parserStmt, prepared, pinfo, res, canAutoCommit)
}
in pkg/sql.(*connExecutor).execStmt
canAutoCommit := ex.implicitTxn() && (tcmd.LastInBatch || !implicitTxnForBatch)
ev, payload, err = ex.execStmt(
ctx, tcmd.Statement, nil /* prepared */, nil /* pinfo */, stmtRes, canAutoCommit,
in pkg/sql.(*connExecutor).execCmd.func1
return err
}()
// Note: we write to ex.statsCollector.PhaseTimes, instead of ex.phaseTimes,
in pkg/sql.(*connExecutor).execCmd
var err error
if err = ex.execCmd(); err != nil {
if errors.IsAny(err, io.EOF, errDrainingComplete) {
in pkg/sql.(*connExecutor).run
}()
return h.ex.run(ctx, s.pool, reserved, cancel)
}
in pkg/sql.(*Server).ServeConn
reservedOwned = false // We're about to pass ownership away.
retErr = sqlServer.ServeConn(ctx, connHandler, reserved, cancelConn)
}()
in pkg/sql/pgwire.(*conn).processCommandsAsync.func1
GOROOT/src/runtime/asm_amd64.s#L1580-L1582 in runtime.goexit

pkg/sql/catalog/internal/validate/validate.go in pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).decorate at line 220
pkg/sql/catalog/internal/validate/validate.go in pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).Report at line 149
pkg/sql/catalog/typedesc/type_desc.go in pkg/sql/catalog/typedesc.(*immutable).ValidateCrossReferences at line 689
pkg/sql/catalog/internal/validate/validate.go in pkg/sql/catalog/internal/validate.Validate.func2 at line 77
pkg/sql/catalog/internal/validate/validate.go in pkg/sql/catalog/internal/validate.(*validationErrorAccumulator).validateDescriptorsAtLevel at line 165
pkg/sql/catalog/internal/validate/validate.go in pkg/sql/catalog/internal/validate.Validate at line 72
pkg/sql/catalog/descs/validate.go in pkg/sql/catalog/descs.(*Collection).Validate at line 38
pkg/sql/catalog/descs/validate.go in pkg/sql/catalog/descs.(*Collection).ValidateUncommittedDescriptors at line 61
pkg/sql/conn_executor_exec.go in pkg/sql.(*connExecutor).commitSQLTransactionInternal at line 1014
pkg/sql/conn_executor_exec.go in pkg/sql.(*connExecutor).commitSQLTransaction at line 950
pkg/sql/conn_executor_exec.go in pkg/sql.(*connExecutor).execStmtInOpenState at line 582
pkg/sql/conn_executor_exec.go in pkg/sql.(*connExecutor).execStmt at line 179
pkg/sql/conn_executor.go in pkg/sql.(*connExecutor).execCmd.func1 at line 1891
pkg/sql/conn_executor.go in pkg/sql.(*connExecutor).execCmd at line 1895
pkg/sql/conn_executor.go in pkg/sql.(*connExecutor).run at line 1817
pkg/sql/conn_executor.go in pkg/sql.(*Server).ServeConn at line 763
pkg/sql/pgwire/conn.go in pkg/sql/pgwire.(*conn).processCommandsAsync.func1 at line 728
GOROOT/src/runtime/asm_amd64.s in runtime.goexit at line 1581
Tag Value
Cockroach Release v22.1.11
Cockroach SHA: 22fa382
Platform linux amd64
Distribution CCL
Environment v22.1.11
Command start-single-node
Go Version ``
# of CPUs
# of Goroutines

Jira issue: CRDB-21783

@cockroach-teamcity cockroach-teamcity added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. O-sentry Originated from an in-the-wild panic report. labels Nov 24, 2022
@yuzefovich
Copy link
Member

dup of #91645

@exalate-issue-sync exalate-issue-sync bot changed the title sentry: type_desc.go:689: type × (111): referencing table 113 was dropped without dependency unlinking (1) keys: [sql.schema.validation_errors.write.cross_references.type sql.schema.validation_errors.write.cr... sentry: type_desc.go:689: type × (111): referencing table 113 was dropped without dependency unlinking (1) keys: [sql.schema.validation_errors.write.cross_references.type sql.schema.validation_errors.write.cr... Dec 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. O-sentry Originated from an in-the-wild panic report.
Projects
None yet
Development

No branches or pull requests

2 participants