You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2020/07/24 00:42:31 UTC

[GitHub] [incubator-pinot] mayankshriv commented on pull request #5745: Allow serving of offline segments immediately/inclusively.

mayankshriv commented on pull request #5745:
URL: https://github.com/apache/incubator-pinot/pull/5745#issuecomment-663299681


   @ianvkoeppe I am unclear on the PR description:
   1. The first offline segment should immediately push the high-water-mark.
   2. Your requirement seems contrary to the other requests we are getting, where the high-water-mark being pushed up pre-maturely (before all segments from the new date are ONLINE) causes inconsistent results during the push time. And we are getting multiple requests to address the inconsistency by deferring the push of high-water-mark until the entire push is complete.


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

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