Bug fix for initGroupBy

zachmu · zachmu · commit f30849b5ca1d · 2025-05-23T14:18:43.000-07:00
diff --git a/sql/planbuilder/scope.go b/sql/planbuilder/scope.go
@@ -224,7 +224,7 @@ func (s *scope) initProc() {
 // initGroupBy creates a container scope for aggregation
 // functions and function inputs.
 func (s *scope) initGroupBy() {
-	if s.groupBy != nil {
+	if s.groupBy == nil {
 		s.groupBy = &groupBy{outScope: s.replace()}
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -224,7 +224,7 @@ func (s *scope) initProc() {`
`224`	`224`	`// initGroupBy creates a container scope for aggregation`
`225`	`225`	`// functions and function inputs.`
`226`	`226`	`func (s *scope) initGroupBy() {`
`227`		`- if s.groupBy != nil {`
	`227`	`+ if s.groupBy == nil {`
`228`	`228`	`s.groupBy = &groupBy{outScope: s.replace()}`
`229`	`229`	`}`
`230`	`230`	`}`