@@ -587,8 +587,8 @@ private Aggregate resolveAggregate(Aggregate aggregate, List<Attribute> children
587587
588588 List <NamedExpression > newAggregates = new ArrayList <>();
589589 // If the groupings are not resolved, skip the resolution of the references to groupings in the aggregates, resolve the
590- // aggregations besides groupings, so that the fields and attributes referenced by the aggregations can be resolved, and
591- // verifier doesn't report field/reference/column not found errors for them.
590+ // aggregations that do not reference to groupings, so that the fields/ attributes referenced by the aggregations can be
591+ // resolved, and verifier doesn't report field/reference/column not found errors for them.
592592 boolean groupingResolved = Resolvables .resolved (groupings );
593593 int size = groupingResolved ? aggregates .size () : aggregates .size () - groupings .size ();
594594 for (int i = 0 ; i < aggregates .size (); i ++) {
@@ -603,10 +603,10 @@ private Aggregate resolveAggregate(Aggregate aggregate, List<Attribute> children
603603 ne = maybeResolved ;
604604 }
605605 } else {
606- // an item in aggregations can reference to groupings explicitly, is groupings are not resolved yet and
606+ // An item in aggregations can reference to groupings explicitly, if groupings are not resolved yet and
607607 // maybeResolved is not resolved, return the original UnresolvedAttribute, so that it has a another chance
608- // to get resolved. For example, {@code }
609- // STATS c = count(emp_no), x = d::int + 1 BY d = (date == "2025-01-01")
608+ // to get resolved in the next iteration.
609+ // For example STATS c = count(emp_no), x = d::int + 1 BY d = (date == "2025-01-01")
610610 if (maybeResolved instanceof Unresolvable == false ) {
611611 changed .set (true );
612612 ne = maybeResolved ;
0 commit comments