ankitsultana commented on code in PR #15743:
URL: https://github.com/apache/pinot/pull/15743#discussion_r2079995355
##########
pinot-query-planner/src/test/resources/queries/PhysicalOptimizerPlans.json:
##########
@@ -421,5 +421,104 @@
]
}
]
+ },
+ "physical_opt_lite_mode_single_rel_queries": {
+ "queries": [
+ {
+ "description": "Simple SELECT with WHERE query.",
+ "sql": "SET usePhysicalOptimizer=true; SET useLiteMode=true; EXPLAIN
PLAN FOR SELECT col2, col3 FROM a WHERE col1 = 'foo'",
+ "output": [
+ "Execution Plan",
+ "\nPhysicalSort(fetch=[100000])",
+ "\n PhysicalProject(col2=[$1], col3=[$2])",
+ "\n PhysicalFilter(condition=[=($0, _UTF-8'foo')])",
+ "\n PhysicalTableScan(table=[[default, a]])",
+ "\n"
+ ]
+ },
+ {
+ "description": "Auto elimination of partial aggregate when group-by on
partitioning column. There's no sort because the limit is added to Agg.",
Review Comment:
I'll raise a small change to update the `explainTerms` of PhysicalAggregate
to match that of `PinotLogicalAggregate`. Didn't do it in this PR because it
will change quite a lot of tests in this json file.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
