gortiz opened a new pull request, #17668: URL: https://github.com/apache/pinot/pull/17668
## Summary This PR introduces a new mode for query logging when the broker query log rate limit is reached. The behavior is controlled by the `broker.query.log.on.rate.limit` configuration property, which now supports a new value: `debug`. The default remains `drop`. ## Motivation Previously, when the rate limit was reached, QueryLogger would drop query logs entirely. This made it difficult to capture and forward all query logs for further analysis or external processing (e.g., to separate files, syslog, or other appenders). ## Changes * Added support for a new value debug for the broker.query.log.on.rate.limit property. * When set to debug, logs are emitted at the INFO level until the rate limit is reached, after which they are logged at the DEBUG level instead of being dropped. * Any value other than debug retains the existing behavior (logs are dropped after the rate limit is reached). ## Impact * Backward compatible: the default remains drop, so existing deployments are unaffected unless the new mode is explicitly enabled. * Provides more flexibility for operators to capture and route all query logs, even under high load. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
