Jackie-Jiang commented on issue #6679:
URL: 
https://github.com/apache/incubator-pinot/issues/6679#issuecomment-878687725


   @mcvsubbu I don't think that the case at least based on my reading. Once 
controller detects a new partition, it will fetch the partition offset based on 
the offset criteria from the stream config (see `StreamMetadataProvider`). Note 
that controller checks new partitions periodically, so at the time new 
partition is detected, the LATEST might not include the earliest event.
   Also, I don't see any drawback of allowing user to set the offset criteria 
for the repair. If not specified, it will fall back to the table config 
setting. So basically as easy to use and more flexible.


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