cube2222
diff --git a/‎logical/logical.go‎
Lines changed: 2 additions & 0 deletions b/‎logical/logical.go‎
Lines changed: 2 additions & 0 deletions
diff --git a/‎…timizer/mark_unused_datasource_fields.go‎ ‎…mizer/remove_unused_datasource_fields.go‎optimizer/mark_unused_datasource_fields.go renamed to optimizer/remove_unused_datasource_fields.go b/‎…timizer/mark_unused_datasource_fields.go‎ ‎…mizer/remove_unused_datasource_fields.go‎optimizer/mark_unused_datasource_fields.go renamed to optimizer/remove_unused_datasource_fields.go
@@ -368,6 +368,8 @@ func (c *Coalesce) Typecheck(ctx context.Context, env physical.Environment, logi
 		outputType = octosql.TypeSum(outputType, expr.Type)
 	}
 
+	// TODO: If any argument is non-nullable, then it should break the for loop and return the current union type as non-nullable.
+
 	return physical.Expression{
 		Type:           outputType,
 		ExpressionType: physical.ExpressionTypeCoalesce,
Original file line number	Diff line number	Diff line change
`@@ -368,6 +368,8 @@ func (c *Coalesce) Typecheck(ctx context.Context, env physical.Environment, logi`
`368`	`368`	`outputType = octosql.TypeSum(outputType, expr.Type)`
`369`	`369`	`}`
`370`	`370`
	`371`	`+ // TODO: If any argument is non-nullable, then it should break the for loop and return the current union type as non-nullable.`
	`372`	`+`
`371`	`373`	`return physical.Expression{`
`372`	`374`	`Type: outputType,`
`373`	`375`	`ExpressionType: physical.ExpressionTypeCoalesce,`