gortiz commented on PR #16080: URL: https://github.com/apache/pinot/pull/16080#issuecomment-2969264781
I'm fine with this PR but I would suggest to include the beta prefix, given this is something we may want to change and deprecate in the future. We are discussing how to apply this limit locally (per server) instead of per broker. Additionally, in the future, we should utilize custom coroutines or virtual threads instead of hundreds of native threads, which are always problematic -- 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
