gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1738408737
##########
pinot-spi/src/main/java/org/apache/pinot/spi/utils/CommonConstants.java:
##########
@@ -1192,6 +1209,9 @@ public enum WindowOverFlowMode {
public static class PlanVersions {
public static final int V1 = 1;
}
+
+ public static final String ASK_SERVERS_FOR_EXPLAIN_PLAN =
"pinot.query.explain.ask.servers";
+ public static final boolean DEFAULT_ASK_SERVERS_FOR_EXPLAIN_PLAN = true;
Review Comment:
Probably this should be changed to false in the initial version. I'm keeping
it as true for a while to verify that all tests are running.
--
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]
