This is an automated email from the ASF dual-hosted git repository. xiangfu0 pushed a commit to branch codex/test-audit-restore-groupby-optimizer-test in repository https://gitbox.apache.org/repos/asf/pinot.git
commit c6cb7f75b0782a3f09297bfa68f99b46a1ccf3c7 Author: Xiang Fu <[email protected]> AuthorDate: Sun Oct 18 23:16:09 2026 -0700 Restore grouped aggregation optimizer coverage --- .../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]
