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: datum.go:1228: expected *DString, found tree.dNull (1) attached stack trace -- stack trace: | runtime.gopanic | GOROOT/src/runtime/panic.go:884 | [...repeated from below...] Wraps: (2) assert... #93429

Closed
cockroach-teamcity opened this issue Dec 12, 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 Dec 12, 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/3803486409/?referrer=webhooks_plugin

Panic message:

datum.go:1228: expected *DString, found tree.dNull
(1) attached stack trace
-- stack trace:
| runtime.gopanic
| GOROOT/src/runtime/panic.go:884
| [...repeated from below...]
Wraps: (2) assertion failure
Wraps: (3) attached stack trace
-- stack trace:
| github.com/cockroachdb/cockroach/pkg/sql/sem/tree.MustBeDString
| github.com/cockroachdb/cockroach/pkg/sql/sem/tree/datum.go:1228
| github.com/cockroachdb/cockroach/pkg/sql.glob..func244
| github.com/cockroachdb/cockroach/pkg/sql/pg_catalog.go:3431
| github.com/cockroachdb/cockroach/pkg/sql.(*virtualDefEntry).getPlanInfo.func1.1
| github.com/cockroachdb/cockroach/pkg/sql/virtual_schema.go:572
| github.com/cockroachdb/cockroach/pkg/sql.setupGenerator.func3
| github.com/cockroachdb/cockroach/pkg/sql/virtual_table.go:127
| github.com/cockroachdb/cockroach/pkg/util/stop.(*Stopper).RunAsyncTaskEx.func2
| github.com/cockroachdb/cockroach/pkg/util/stop/stopper.go:489
| runtime.goexit
| GOROOT/src/runtime/asm_amd64.s:1594
Wraps: (4) expected *DString, found tree.dNull
Error types: (1) *withstack.withStack (2) *assert.withAssertionFailure (3) *withstack.withStack (4) *errutil.leafError
-- report composition:
*errutil.leafError: expected *DString, found tree.dNull
datum.go:1228: *withstack.withStack (top exception)
*assert.withAssertionFailure
panic.go:884: *withstack.withStack (1)
(check the extra data payloads)

Stacktrace (expand for inline code snippets):

GOROOT/src/runtime/panic.go#L883-L885 in runtime.gopanic

if !ok {
panic(errors.AssertionFailedf("expected *DString, found %T", e))
}
in pkg/sql/sem/tree.MustBeDString
statisticsID := tree.MustBeDInt(row[0])
name := tree.MustBeDString(row[1])
tableID := tree.MustBeDInt(row[2])
in pkg/sql.glob..func244
generator, cleanup, setupError := setupGenerator(ctx, func(ctx context.Context, pusher rowPusher) error {
return def.populate(ctx, p, dbDesc, func(row ...tree.Datum) error {
if err := e.validateRow(row, columns); err != nil {
in pkg/sql.(*virtualDefEntry).getPlanInfo.func1.1
}
err := worker(ctx, funcRowPusher(addRow))
// If the query was canceled, next() will already return a
in pkg/sql.setupGenerator.func3
sp.UpdateGoroutineIDToCurrent()
f(ctx)
}()
in pkg/util/stop.(*Stopper).RunAsyncTaskEx.func2
GOROOT/src/runtime/asm_amd64.s#L1593-L1595 in runtime.goexit

GOROOT/src/runtime/panic.go in runtime.gopanic at line 884
pkg/sql/sem/tree/datum.go in pkg/sql/sem/tree.MustBeDString at line 1228
pkg/sql/pg_catalog.go in pkg/sql.glob..func244 at line 3431
pkg/sql/virtual_schema.go in pkg/sql.(*virtualDefEntry).getPlanInfo.func1.1 at line 572
pkg/sql/virtual_table.go in pkg/sql.setupGenerator.func3 at line 127
pkg/util/stop/stopper.go in pkg/util/stop.(*Stopper).RunAsyncTaskEx.func2 at line 489
GOROOT/src/runtime/asm_amd64.s in runtime.goexit at line 1594
Tag Value
Cockroach Release v22.2.0
Cockroach SHA: 77667a1
Platform linux amd64
Distribution CCL
Environment v22.2.0
Command demo
Go Version ``
# of CPUs
# of Goroutines

Jira issue: CRDB-22330

@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 Dec 12, 2022
@jordanlewis
Copy link
Member

This was me and it's a duplicate of #93430.

@exalate-issue-sync exalate-issue-sync bot changed the title sentry: datum.go:1228: expected *DString, found tree.dNull (1) attached stack trace -- stack trace: | runtime.gopanic | GOROOT/src/runtime/panic.go:884 | [...repeated from below...] Wraps: (2) assert... sentry: datum.go:1228: expected *DString, found tree.dNull (1) attached stack trace -- stack trace: | runtime.gopanic | GOROOT/src/runtime/panic.go:884 | [...repeated from below...] Wraps: (2) assert... Dec 12, 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