albertobastos commented on code in PR #15078:
URL: https://github.com/apache/pinot/pull/15078#discussion_r1959144561
##########
pinot-spi/src/main/java/org/apache/pinot/spi/utils/CommonConstants.java:
##########
@@ -704,6 +708,9 @@ public enum Type {
}
public static final String PREFIX_OF_CONFIG_OF_PINOT_FS_FACTORY =
"pinot.broker.storage.factory";
+
Review Comment:
Done!
266af9cb30
##########
pinot-spi/src/main/java/org/apache/pinot/spi/utils/CommonConstants.java:
##########
@@ -584,6 +584,10 @@ public static class QueryOptionKey {
// Custom Query ID provided by the client
public static final String CLIENT_QUERY_ID = "clientQueryId";
+
+ // Use MSQE compiler when trying to fill a response with no schema
metadata
+ // (overrides the "pinot.broker.use.msqe.schema" broker conf)
+ public static final String USE_MSQE_EMPTY_SCHEMA =
"useMsqeWhenEmptySchema";
Review Comment:
Done!
266af9cb30
##########
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseSingleStageBrokerRequestHandler.java:
##########
@@ -904,6 +909,18 @@ protected BrokerResponse doHandleRequest(long requestId,
String query, SqlNodeAn
return brokerResponse;
}
+ /**
+ * Determines if MSQE can be used to try to fill an empty schema response.
+ */
+ private boolean useMSQEToFillEmptySchema(@Nullable PinotQuery pinotQuery) {
Review Comment:
> Will `pinotQuery` ever be `null`?
Actually no, just thought it was used somewhere else when I coded it.
Annotation removed.
> You may put this logic into `QueryOptionsUtils`, similar to
`isUseFixedReplica()`
I moved it with the addition of a "default value" so I can use it directly
from the broker handler.
266af9cb30
--
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
