walterddr commented on code in PR #11052:
URL: https://github.com/apache/pinot/pull/11052#discussion_r1266942000
##########
pinot-query-planner/src/test/java/org/apache/pinot/query/QueryEnvironmentTestBase.java:
##########
@@ -202,4 +205,10 @@ public String toString() {
}
}
}
+
+ // temporary function to strip the Physical plan from the explain query plan.
+ // physical plan is present for the explain queries by default.
Review Comment:
ok now I understand the problem.. sorry for the confusion, it seems like we
should set the default to LOGICAL instead of PHYSICAL
--
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]
