Jackie-Jiang commented on issue #7338: URL: https://github.com/apache/pinot/issues/7338#issuecomment-994032880
@suddendust Currently we only acquire one semaphore for `reloadAllSegments()` because internally it will load segments in sequential. When we make it using multiple threads, we should also acquire more semaphores. `reload` and `refresh` are essentially doing the same thing, where we need to keep 2 copies of segment loaded at the same time, so we should use the same semaphore to control both of them to avoid taking too much resource -- 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
