[GitHub] [pinot] sajjad-moradi edited a comment on pull request #6291: Add consumption rate limiter for LLConsumer

[GitBox <gi...@apache.org>]

sajjad-moradi edited a comment on pull request #6291:
URL: https://github.com/apache/pinot/pull/6291#issuecomment-1022799793


   > Then why do we have a 10-minute cache refresh for the number of partitions? Just fetch it once and set the rate limiter right?
   
   Fetching-it-once works for current consuming segments, but it'll be problematic for the future consuming segments in case the number of partitions changes.
   The cache is used to prevent fetching the num partitions for multiple consuming segments of the same topic on the same host.


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