jugomezv commented on PR #10418:
URL: https://github.com/apache/pinot/pull/10418#issuecomment-1471328453

   > I don't think this is correct. Imagine we got some real messages, followed 
by some filtered messages, followed by some real messages. We should not set 
the delay to 0 in case all messages are filtered because it doesn't indicate we 
have already caught up. Am I missing something here?
   
   @Jackie-Jiang I see your point of view, but the situation I am trying to 
address here is constant repeated invocations of the loop where we filtered all 
messages, the current outcome is not correct correct either: an event comes in 
and we record that delay, then event batches are all filtered: the current 
outcome is time continues to ramp up to hours and hours and we are actually 
consuming. This is confusing and not correct. Do you have a better suggestion?
   
   We have seen this in some tables that are not actively being used and I 
think someone else reported earlier.


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

Reply via email to