added explanation comment

angelamayxie · angelamayxie · commit 17278ed82987 · 2025-10-15T16:39:17.000-07:00
diff --git a/sql/analyzer/validation_rules.go b/sql/analyzer/validation_rules.go
@@ -338,6 +338,8 @@ func validateGroupBy(ctx *sql.Context, a *Analyzer, n sql.Node, scope *plan.Scop
 				}
 			}
 		case *plan.Project:
+			// Project nodes that are direct children of Having nodes include aliases for columns that are part of an
+			// aggregate function that aren't necessarily selected expressions and therefore shouldn't be validated
 			if _, isHaving := parent.(*plan.Having); !isHaving {
 				project = n
 				orderBy = nil

Original file line number	Diff line number	Diff line change
`@@ -338,6 +338,8 @@ func validateGroupBy(ctx sql.Context, a Analyzer, n sql.Node, scope *plan.Scop`
`338`	`338`	`}`
`339`	`339`	`}`
`340`	`340`	`case *plan.Project:`
	`341`	`+ // Project nodes that are direct children of Having nodes include aliases for columns that are part of an`
	`342`	`+ // aggregate function that aren't necessarily selected expressions and therefore shouldn't be validated`
`341`	`343`	`if _, isHaving := parent.(*plan.Having); !isHaving {`
`342`	`344`	`project = n`
`343`	`345`	`orderBy = nil`