rhodo opened a new pull request, #16342:
URL: https://github.com/apache/pinot/pull/16342

   The current logic only consider concurrency from 
`_segmentDownloadSemaphore`, but not `_segmentOperationsThrottler`. Move the 
metrics increment and decrement to very beginning and end right close to 
segment download.
   
   Also we might not need another atomic counter 
`_numSegmentsAcquiredDownloadSemaphore` to keep track of everything actually. 
As the gauge can itself be add or decrement.


-- 
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

Reply via email to