mcvsubbu commented on PR #8986: URL: https://github.com/apache/pinot/pull/8986#issuecomment-1172793016
> > You should also check the idealstate once before entering the consumeLoop() > > That would be for handling a rare edge case. The probability of that case happening is very low and can be calculated as `t1/t2` where `t1`: time duration between receiving online-to-consuming msg and entering the consumeLoop `t2`: consumeLoop duration Note that t1 is in microseconds-milliseconds and t2 is in hours (depending on table config). > > If that case happens, the partition will eventually be paused when after the segment gets completed. The operator can reissue the pause request if they need it sooner. Makes sense. Keeps the first version simpler while kicking the tires. We can enhance this as we hear back regarding its use. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
