noob-se7en opened a new pull request, #16339: URL: https://github.com/apache/pinot/pull/16339
**Problem** RealtimeConsumptionRateManager.MetricEmitter.emitMetric for ServerRateLimiter is not thread safe because serverRateLimiter object is a shared variable and MetricEmitter.emitMetric before this PR relies on shared Mutable state. We can't rely on ServerGauge.CONSUMPTION_QUOTA_UTILIZATION metric because of above. **Solution** Make MetricEmitter.emitMetric thread safe and offloads the metric emission part from the consumer thread. -- 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
