somandal commented on code in PR #10228:
URL: https://github.com/apache/pinot/pull/10228#discussion_r1099622163
##########
pinot-query-planner/src/test/resources/queries/Aggregates.json:
##########
@@ -0,0 +1,34 @@
+{
+ "aggregates_planning_tests": {
Review Comment:
I'll rename the files to include "Plans" in the name for now. I kept the
planning and execution tests separate since the test folders are in separate
modules. Perhaps in the next PR we can create a common tests folder outside the
two modules and combine these tests into a single 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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
