yashmayya commented on code in PR #14938: URL: https://github.com/apache/pinot/pull/14938#discussion_r1934242669
########## pinot-broker/src/main/java/org/apache/pinot/broker/querylog/QueryLogger.java: ########## @@ -255,6 +257,12 @@ void doFormat(StringBuilder builder, QueryLogger logger, QueryLogParams params) builder.append(CommonConstants.UNKNOWN); } } + }, + IS_MULTI_STAGE_QUERY_ENGINE("isMultiStageQueryEngine") { Review Comment: Alternatively, we could call this `queryEngine` with possible options being `singleStage` / `multiStage` so that it is open to future extensions. -- 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