gortiz commented on PR #16075: URL: https://github.com/apache/pinot/pull/16075#issuecomment-2982921299
> I'm curious to know why this should be a query option and how is it intended to be used in practice? If this is only to improve error reporting for query failures, shouldn't we just automatically introduce some additional "passive" timeout internally (maybe based on some small %age of the actual query timeout)? We use a default value of 100 ms, but this can be adjusted with query options. I think that could be useful in production environments where the 100ms is not sufficient (i.e., queries where the stage tree is very deep). But even if it isn't, it is super useful to have this query option to verify the feature works on a quickstart :smile: -- 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
