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: expr.go:148: ReturnType called on TypedExpr with empty typeAnnotation. Was the underlying Expr type-checked before asserting a type of TypedExpr? (1) attached stack trace -- stack trace: |... #136510

Closed
cockroach-sentry opened this issue Dec 2, 2024 · 2 comments
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. T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) X-blathers-triaged blathers was able to find an owner

Comments

@cockroach-sentry
Copy link
Collaborator

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

Sentry Link: https://cockroach-labs.sentry.io/issues/6105893008/?referrer=webhooks_plugin

Panic Message:

expr.go:148: ReturnType called on TypedExpr with empty typeAnnotation. Was the underlying Expr type-checked before asserting a type of TypedExpr?
(1) attached stack trace
  -- stack trace:
  | runtime.gopanic
  | 	GOROOT/src/runtime/panic.go:770
  | [...repeated from below...]
Wraps: (2) assertion failure
Wraps: (3) attached stack trace
  -- stack trace:
  | github.com/cockroachdb/cockroach/pkg/sql/sem/tree.typeAnnotation.assertTyped
  | 	pkg/sql/sem/tree/expr.go:148
  | github.com/cockroachdb/cockroach/pkg/sql/sem/tree.(*Subquery).TypeCheck
  | 	pkg/sql/sem/tree/type_check.go:1764
  | github.com/cockroachdb/cockroach/pkg/sql/sem/tree.TypeCheck
  | 	pkg/sql/sem/tree/type_check.go:345
  | github.com/cockroachdb/cockroach/pkg/sql/sem/tree.TypeCheckAndRequire
  | 	pkg/sql/sem/tree/type_check.go:355
  | github.com/cockroachdb/cockroach/pkg/sql.(*planner).analyzeExpr
  | 	pkg/sql/analyze_expr.go:34
  | github.com/cockroachdb/cockroach/pkg/sql.(*planner).planTenantSpec
  | 	pkg/sql/tenant_spec.go:63
  | github.com/cockroachdb/cockroach/pkg/sql.(*planner).LookupTenantInfo
  | 	pkg/sql/tenant_spec.go:148
  | github.com/cockroachdb/cockroach/pkg/crosscluster/physical.alterReplicationJobHook.func1
  | 	pkg/crosscluster/physical/alter_replication_job.go:220
  | github.com/cockroachdb/cockroach/pkg/sql.(*hookFnNode).startExec.func1
  | 	pkg/sql/planhook.go:194
  | github.com/cockroachdb/cockroach/pkg/util/stop.(*Stopper).RunAsyncTaskEx.func2
  | 	pkg/util/stop/stopper.go:498
  | runtime.goexit
  | 	src/runtime/asm_arm64.s:1222
Wraps: (4) ReturnType called on TypedExpr with empty typeAnnotation. Was the underlying Expr type-checked before asserting a type of TypedExpr?
Error types: (1) *withstack.withStack (2) *assert.withAssertionFailure (3) *withstack.withStack (4) *errutil.leafError
-- report composition:
*errutil.leafError: ReturnType called on TypedExpr with empty typeAnnotation. Was the underlying Expr type-checked before asserting a type of TypedExpr?
expr.go:148: *withstack.withStack (top exception)
*assert.withAssertionFailure
panic.go:770: *withstack.withStack (1)
(check the extra data payloads)
Stacktrace (expand for inline code snippets):

src/runtime/asm_arm64.s#L1221-L1223
pkg/util/stop/stopper.go#L497-L499
pkg/sql/planhook.go#L193-L195
pkg/crosscluster/physical/alter_replication_job.go#L219-L221
pkg/sql/tenant_spec.go#L147-L149
pkg/sql/tenant_spec.go#L62-L64
pkg/sql/analyze_expr.go#L33-L35
pkg/sql/sem/tree/type_check.go#L354-L356
pkg/sql/sem/tree/type_check.go#L344-L346
pkg/sql/sem/tree/type_check.go#L1763-L1765
pkg/sql/sem/tree/expr.go#L147-L149
GOROOT/src/runtime/panic.go#L769-L771
src/runtime/asm_arm64.s#L1221-L1223
pkg/util/stop/stopper.go#L497-L499
pkg/sql/planhook.go#L193-L195
pkg/crosscluster/physical/alter_replication_job.go#L219-L221
pkg/sql/tenant_spec.go#L147-L149
pkg/sql/tenant_spec.go#L62-L64
pkg/sql/analyze_expr.go#L33-L35
pkg/sql/sem/tree/type_check.go#L354-L356
pkg/sql/sem/tree/type_check.go#L344-L346
pkg/sql/sem/tree/type_check.go#L1763-L1765
pkg/sql/sem/tree/expr.go#L147-L149

src/runtime/asm_arm64.s in runtime.goexit at line 1222
pkg/util/stop/stopper.go in pkg/util/stop.(*Stopper).RunAsyncTaskEx.func2 at line 498
pkg/sql/planhook.go in pkg/sql.(*hookFnNode).startExec.func1 at line 194
pkg/crosscluster/physical/alter_replication_job.go in pkg/crosscluster/physical.alterReplicationJobHook.func1 at line 220
pkg/sql/tenant_spec.go in pkg/sql.(*planner).LookupTenantInfo at line 148
pkg/sql/tenant_spec.go in pkg/sql.(*planner).planTenantSpec at line 63
pkg/sql/analyze_expr.go in pkg/sql.(*planner).analyzeExpr at line 34
pkg/sql/sem/tree/type_check.go in pkg/sql/sem/tree.TypeCheckAndRequire at line 355
pkg/sql/sem/tree/type_check.go in pkg/sql/sem/tree.TypeCheck at line 345
pkg/sql/sem/tree/type_check.go in pkg/sql/sem/tree.(*Subquery).TypeCheck at line 1764
pkg/sql/sem/tree/expr.go in pkg/sql/sem/tree.typeAnnotation.assertTyped at line 148
GOROOT/src/runtime/panic.go in runtime.gopanic at line 770
src/runtime/asm_arm64.s in runtime.goexit at line 1222
pkg/util/stop/stopper.go in pkg/util/stop.(*Stopper).RunAsyncTaskEx.func2 at line 498
pkg/sql/planhook.go in pkg/sql.(*hookFnNode).startExec.func1 at line 194
pkg/crosscluster/physical/alter_replication_job.go in pkg/crosscluster/physical.alterReplicationJobHook.func1 at line 220
pkg/sql/tenant_spec.go in pkg/sql.(*planner).LookupTenantInfo at line 148
pkg/sql/tenant_spec.go in pkg/sql.(*planner).planTenantSpec at line 63
pkg/sql/analyze_expr.go in pkg/sql.(*planner).analyzeExpr at line 34
pkg/sql/sem/tree/type_check.go in pkg/sql/sem/tree.TypeCheckAndRequire at line 355
pkg/sql/sem/tree/type_check.go in pkg/sql/sem/tree.TypeCheck at line 345
pkg/sql/sem/tree/type_check.go in pkg/sql/sem/tree.(*Subquery).TypeCheck at line 1764
pkg/sql/sem/tree/expr.go in pkg/sql/sem/tree.typeAnnotation.assertTyped at line 148

Tags

Tag Value
Command demo
Environment development
Go Version go1.22.8 X:nocoverageredesign
Platform darwin arm64
Distribution CCL
Cockroach Release v25.1.0-alpha.00000000-dev-749b563a7b5957b2db487c6e8e2fdeed8630103d
Cockroach SHA 749b563
# of CPUs 10
# of Goroutines 468
@cockroach-sentry cockroach-sentry 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 2, 2024
Copy link

blathers-crl bot commented Dec 2, 2024

CC'ing via the CODEOWNERS-based sentry heuristic:

  • @cockroachdb/sql-foundations

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

@blathers-crl blathers-crl bot added T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) X-blathers-triaged blathers was able to find an owner labels Dec 2, 2024
@fqazi
Copy link
Collaborator

fqazi commented Dec 3, 2024

Duplicate of #136339

@fqazi fqazi marked this as a duplicate of #136339 Dec 3, 2024
@fqazi fqazi closed this as completed Dec 3, 2024
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. T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) X-blathers-triaged blathers was able to find an owner
Projects
None yet
Development

No branches or pull requests

2 participants