This is an automated email from the ASF dual-hosted git repository.
Jackie-Jiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new d234d4e2ad8 Restore grouped aggregation optimizer coverage (#19933)
d234d4e2ad8 is described below
commit d234d4e2ad897842db0b1c56409d98523a094913
Author: Xiang Fu <[email protected]>
AuthorDate: Mon Oct 19 11:23:19 2026 -0700
Restore grouped aggregation optimizer coverage (#19933)
---
.../apache/pinot/sql/parsers/rewriter/AggregationOptimizerTest.java | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
a/pinot-common/src/test/java/org/apache/pinot/sql/parsers/rewriter/AggregationOptimizerTest.java
b/pinot-common/src/test/java/org/apache/pinot/sql/parsers/rewriter/AggregationOptimizerTest.java
index 33e770f00ff..c4335dc24fd 100644
---
a/pinot-common/src/test/java/org/apache/pinot/sql/parsers/rewriter/AggregationOptimizerTest.java
+++
b/pinot-common/src/test/java/org/apache/pinot/sql/parsers/rewriter/AggregationOptimizerTest.java
@@ -339,10 +339,10 @@ public class AggregationOptimizerTest {
}
}
- @Test(enabled = false) // TODO: GROUP BY optimization needs investigation -
currently uses 'values' function
+ @Test
public void testGroupByWithOptimization() {
- // Test: SELECT sum(value + 10) FROM mytable GROUP BY category
- String query = "SELECT sum(value + 10) FROM mytable GROUP BY category";
+ // Quote VALUE so the parser treats it as a column instead of a table
value constructor.
+ String query = "SELECT sum(\"value\" + 10) FROM mytable GROUP BY category";
PinotQuery pinotQuery = CalciteSqlParser.compileToPinotQuery(query);
assertTopLevelOperator(pinotQuery.getSelectList().get(0), "sum");
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
