ianvkoeppe commented on a change in pull request #5745: URL: https://github.com/apache/incubator-pinot/pull/5745#discussion_r460197217
########## File path: pinot-spi/src/main/java/org/apache/pinot/spi/config/table/QueryConfig.java ########## @@ -37,15 +37,23 @@ // If the server times out, it will directly interrupt the query execution. The server response does not matter much // because by the time the server times out, the broker should already timed out and returned the response. private final Long _timeoutMs; + private final Boolean _serveOfflineSegmentsImmediately; Review comment: Not finding where the docs reside within repo, but I've added documentation to the config class. ---------------------------------------------------------------- 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. 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