From e92863224561fdf4e4787a87870b6610f0d6a8d7 Mon Sep 17 00:00:00 2001 From: Jason Fulghum Date: Thu, 5 Jun 2025 15:59:55 -0700 Subject: [PATCH] Simplifying logic for hasSingleOutput --- sql/analyzer/apply_hash_in.go | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/sql/analyzer/apply_hash_in.go b/sql/analyzer/apply_hash_in.go index f89334ae0e..b51e2378e3 100644 --- a/sql/analyzer/apply_hash_in.go +++ b/sql/analyzer/apply_hash_in.go @@ -56,16 +56,13 @@ func applyHashIn(ctx *sql.Context, a *Analyzer, n sql.Node, scope *plan.Scope, s // hasSingleOutput checks if an expression evaluates to a single output func hasSingleOutput(e sql.Expression) bool { - return !transform.InspectExpr(e, func(expr sql.Expression) bool { + return transform.InspectExpr(e, func(expr sql.Expression) bool { switch expr.(type) { - case expression.Tuple, *expression.Literal, *expression.GetField, - expression.Comparer, *expression.Convert, sql.FunctionExpression, - *expression.IsTrue, *expression.IsNull, expression.ArithmeticOp: + case *plan.Subquery: return false default: return true } - return false }) }