sentry: exec_factory_util.go:44: unexpected node type <nil> -- *errutil.leafError: unexpected node type <nil> (1) exec_factory_util.go:44: *withstack.withStack (top exception) *assert.withAssertionFailure conn_executor.go:506: *withstack.withStack (2) *safedetails.withSafeDetails: while executing: ALTER DATABASE _ OWNER TO _ (3) conn_executor.go:506: *withstack.withStack (4) (check the extra data payloads)

[cockroach-teamcity]

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/2072211085/?referrer=webhooks_plugin

Panic message:

exec_factory_util.go:44: unexpected node type
--
*errutil.leafError: unexpected node type (1)
exec_factory_util.go:44: *withstack.withStack (top exception)
*assert.withAssertionFailure
conn_executor.go:506: *withstack.withStack (2)
*safedetails.withSafeDetails: while executing: ALTER DATABASE _ OWNER TO _ (3)
conn_executor.go:506: *withstack.withStack (4)
(check the extra data payloads)

Stacktrace (expand for inline code snippets):

cockroach/pkg/sql/conn_executor.go

Lines 505 to 507 in 92d9495

	r := recover()
	h.ex.closeWrapper(ctx, r)
	}()

in pkg/sql.(*Server).ServeConn.func1
/usr/local/go/src/runtime/panic.go#L678-L680 in runtime.gopanic

cockroach/pkg/sql/exec_factory_util.go

Lines 43 to 45 in 92d9495

	default:
	panic(errors.AssertionFailedf("unexpected node type %T", node))
	}

in pkg/sql.constructPlan.func1

cockroach/pkg/sql/exec_factory_util.go

Lines 46 to 48 in 92d9495

	}
	assignPlan(&res.main, root)
	if len(subqueries) > 0 {

in pkg/sql.constructPlan

cockroach/pkg/sql/opt_exec_factory.go

Lines 1025 to 1027 in 92d9495

	}
	return constructPlan(ef.planner, root, subqueries, cascades, checks)
	}

in pkg/sql.(*execFactory).ConstructPlan

cockroach/pkg/sql/opt/exec/explain/explain_factory.go

Lines 150 to 152 in 92d9495

	var err error
	p.WrappedPlan, err = f.wrappedFactory.ConstructPlan(
	p.Root.WrappedNode(), wrappedSubqueries, wrappedCascades, wrappedChecks,

in pkg/sql/opt/exec/explain.(*Factory).ConstructPlan

cockroach/pkg/sql/opt/exec/execbuilder/builder.go

Lines 140 to 142 in 92d9495

	}
	return b.factory.ConstructPlan(plan.root, b.subqueries, b.cascades, b.checks)
	}

in pkg/sql/opt/exec/execbuilder.(*Builder).Build

cockroach/pkg/sql/plan_opt.go

Lines 580 to 582 in 92d9495

	bld := execbuilder.New(explainFactory, mem, &opc.catalog, mem.RootExpr(), evalCtx, allowAutoCommit)
	plan, err := bld.Build()
	if err != nil {

in pkg/sql.(*optPlanningCtx).runExecBuilder

cockroach/pkg/sql/plan_opt.go

Lines 264 to 266 in 92d9495

	// If we got here, we did not create a plan above.
	return opc.runExecBuilder(
	&p.curPlan,

in pkg/sql.(*planner).makeOptimizerPlan

cockroach/pkg/sql/conn_executor_exec.go

Lines 900 to 902 in 92d9495

	if err := planner.makeOptimizerPlan(ctx); err != nil {
	log.VEventf(ctx, 1, "optimizer plan failed: %v", err)

in pkg/sql.(*connExecutor).makeExecPlan

cockroach/pkg/sql/conn_executor_exec.go

Lines 779 to 781 in 92d9495

	// between here and there needs to happen even if there's an error.
	err := ex.makeExecPlan(ctx, planner)
	// We'll be closing the plan manually below after execution; this

in pkg/sql.(*connExecutor).dispatchToExecutionEngine

cockroach/pkg/sql/conn_executor_exec.go

Lines 638 to 640 in 92d9495

	p.autoCommit = os.ImplicitTxn.Get() && !ex.server.cfg.TestingKnobs.DisableAutoCommit
	if err := ex.dispatchToExecutionEngine(ctx, p, res); err != nil {
	return nil, nil, err

in pkg/sql.(*connExecutor).execStmtInOpenState

cockroach/pkg/sql/conn_executor_exec.go

Lines 113 to 115 in 92d9495

	} else {
	ev, payload, err = ex.execStmtInOpenState(ctx, stmt, res, pinfo)
	}

in pkg/sql.(*connExecutor).execStmt

cockroach/pkg/sql/conn_executor.go

Lines 1464 to 1466 in 92d9495

	stmtCtx := withStatement(ctx, ex.curStmt)
	ev, payload, err = ex.execStmt(stmtCtx, curStmt, stmtRes, nil /* pinfo */)
	return err

in pkg/sql.(*connExecutor).execCmd.func1

cockroach/pkg/sql/conn_executor.go

Lines 1466 to 1468 in 92d9495

	return err
	}()
	// Note: we write to ex.statsCollector.phaseTimes, instead of ex.phaseTimes,

in pkg/sql.(*connExecutor).execCmd

cockroach/pkg/sql/conn_executor.go

Lines 1390 to 1392 in 92d9495

	var err error
	if err = ex.execCmd(ex.Ctx()); err != nil {
	if errors.IsAny(err, io.EOF, errDrainingComplete) {

in pkg/sql.(*connExecutor).run

cockroach/pkg/sql/conn_executor.go

Lines 507 to 509 in 92d9495

	}()
	return h.ex.run(ctx, s.pool, reserved, cancel)
	}

in pkg/sql.(*Server).ServeConn

cockroach/pkg/sql/pgwire/conn.go

Lines 625 to 627 in 92d9495

	reservedOwned = false // We're about to pass ownership away.
	retErr = sqlServer.ServeConn(ctx, connHandler, reserved, cancelConn)
	}()

in pkg/sql/pgwire.(*conn).processCommandsAsync.func1
/usr/local/go/src/runtime/asm_amd64.s#L1356-L1358 in runtime.goexit

pkg/sql/conn_executor.go in pkg/sql.(*Server).ServeConn.func1 at line 506
/usr/local/go/src/runtime/panic.go in runtime.gopanic at line 679
pkg/sql/exec_factory_util.go in pkg/sql.constructPlan.func1 at line 44
pkg/sql/exec_factory_util.go in pkg/sql.constructPlan at line 47
pkg/sql/opt_exec_factory.go in pkg/sql.(*execFactory).ConstructPlan at line 1026
pkg/sql/opt/exec/explain/explain_factory.go in pkg/sql/opt/exec/explain.(*Factory).ConstructPlan at line 151
pkg/sql/opt/exec/execbuilder/builder.go in pkg/sql/opt/exec/execbuilder.(*Builder).Build at line 141
pkg/sql/plan_opt.go in pkg/sql.(*optPlanningCtx).runExecBuilder at line 581
pkg/sql/plan_opt.go in pkg/sql.(*planner).makeOptimizerPlan at line 265
pkg/sql/conn_executor_exec.go in pkg/sql.(*connExecutor).makeExecPlan at line 901
pkg/sql/conn_executor_exec.go in pkg/sql.(*connExecutor).dispatchToExecutionEngine at line 780
pkg/sql/conn_executor_exec.go in pkg/sql.(*connExecutor).execStmtInOpenState at line 639
pkg/sql/conn_executor_exec.go in pkg/sql.(*connExecutor).execStmt at line 114
pkg/sql/conn_executor.go in pkg/sql.(*connExecutor).execCmd.func1 at line 1465
pkg/sql/conn_executor.go in pkg/sql.(*connExecutor).execCmd at line 1467
pkg/sql/conn_executor.go in pkg/sql.(*connExecutor).run at line 1391
pkg/sql/conn_executor.go in pkg/sql.(*Server).ServeConn at line 508
pkg/sql/pgwire/conn.go in pkg/sql/pgwire.(*conn).processCommandsAsync.func1 at line 626
/usr/local/go/src/runtime/asm_amd64.s in runtime.goexit at line 1357

Tag	Value
Cockroach Release	v20.2.2
Cockroach SHA:	92d9495
Platform	linux amd64
Distribution	CCL
Environment	v20.2.2
Command	server
Go Version	``
# of CPUs
# of Goroutines

[jordanlewis]

Dupe of #56861