You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2021/11/25 00:37:11 UTC

[GitHub] [pinot] richardstartin commented on pull request #7828: Fix thread safety issue and add cache to EmptySegmentPruner

richardstartin commented on pull request #7828:
URL: https://github.com/apache/pinot/pull/7828#issuecomment-978582467


   Why not use a `Sets.newConcurrentHashSet()` rather than copy on write? That would simplify this a lot.


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org