yashmayya opened a new pull request, #15724: URL: https://github.com/apache/pinot/pull/15724
- https://github.com/apache/pinot/pull/15189 added this to keep track of the estimated number of query server threads from each broker's point of view. - The methods where the variable is incremented / decremented can be called from multiple query request handling threads concurrently and it's not thread safe to use a regular int. - This patch uses an atomic variable instead for thread-safe computation. Note that this bug didn't affect the throttling logic itself since the variable is only used for the gauge metric computation. -- 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
