cockroachdb / cockroach

CockroachDB — the cloud native, distributed SQL database designed for high availability, effortless scale, and control over data placement.
https://www.cockroachlabs.com
Other
29.88k stars 3.77k forks source link

sql: nil pointer exception in `populateTxnExecutionInsights` #129936

Closed mgartner closed 2 days ago

mgartner commented 2 weeks ago

Stack track:

* ERROR: a panic has occurred!
* runtime error: invalid memory address or nil pointer dereference
* (1) attached stack trace
*   -- stack trace:
*   | runtime.gopanic
*   |   GOROOT/src/runtime/panic.go:770
*   | runtime.panicmem
*   |   GOROOT/src/runtime/panic.go:261
*   | runtime.sigpanic
*   |   GOROOT/src/runtime/signal_unix.go:881
*   | github.com/cockroachdb/cockroach/pkg/sql.populateStmtInsights
*   |   github.com/cockroachdb/cockroach/pkg/sql/crdb_internal.go:8462
*   | github.com/cockroachdb/cockroach/pkg/sql.init.func735
*   |   github.com/cockroachdb/cockroach/pkg/sql/crdb_internal.go:8412
*   | github.com/cockroachdb/cockroach/pkg/sql.(*virtualDefEntry).getPlanInfo.func1.1
*   |   github.com/cockroachdb/cockroach/pkg/sql/virtual_schema.go:661
*   | 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:484
*   | runtime.goexit
*   |   src/runtime/asm_arm64.s:1222
* Wraps: (2) runtime error: invalid memory address or nil pointer dereference
* Error types: (1) *withstack.withStack (2) runtime.errorString
*

The NPE is occurring here: https://github.com/cockroachdb/cockroach/blob/15db309a564809b9e1de1b832ee13b2e0c80f237/pkg/sql/crdb_internal.go#L8263

Jira issue: CRDB-41783

blathers-crl[bot] commented 2 weeks ago

Hi @mgartner, please add branch-* labels to identify which branch(es) this C-bug affects.

:owl: Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

srosenberg commented 6 days ago

Potentially caused by the same data race described in 130290.