yashmayya commented on code in PR #14275:
URL: https://github.com/apache/pinot/pull/14275#discussion_r1810770763
##########
pinot-server/src/main/java/org/apache/pinot/server/starter/ServerInstance.java:
##########
@@ -125,9 +125,14 @@ public ServerInstance(ServerConf serverConf, HelixManager
helixManager, AccessCo
if (serverConf.isMultiStageServerEnabled()) {
LOGGER.info("Initializing Multi-stage query engine");
+ if (serverConf.isMultiStageTlsServerEnabled()) {
+ // TODO: Dispatch clients cannot be configured to use TLS, so using
TLS for multi-stage query engine is not
Review Comment:
```suggestion
// TODO: Dispatch clients currently can't be configured to use TLS,
so using TLS for multi-stage query engine is not
```
nit: the existing wording makes it sound like there's some technical
limitation rather than it just being an implementation TODO
--
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
