61yao opened a new pull request, #10350: URL: https://github.com/apache/pinot/pull/10350
New segments won't exclude unavailable instances from replica group. Also, we don't report new segments as unavailable segments. Definition of new segments: segments with creation age more than 5 minutes and whose external view hasn't converged with ideal state. Implication of this implementation: 1) Inconsistency issues across server when querying We think this inconsistency is acceptable because it greatly improves query availability. 2) Skip reporting segments as error if any new segment goes to error state. This is also acceptable because if the segment doesn’t resume to a healthy state within 5 minutes, we will treat this as an error. However, since we only update new segments upon assignment/instance change, some of the new segments may expire under clock and we are not able to track them. So we may skip reporting issues for these segments. Details can be seen in: https://docs.google.com/document/d/1nfNZxVOXMIykabsC73Ab83SnHrNaZrlLRmr3UkFTE_c/edit#heading=h.ze4g3qr2kl2v -- 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
