[GitHub] [pinot] mcvsubbu commented on pull request #8986: Support pause/resume consumption of realtime tables

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

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