tibrewalpratik17 commented on issue #15846: URL: https://github.com/apache/pinot/issues/15846#issuecomment-2946091455
> Another approach can be using seqId to break the tie, as the seqId field is encoded in LLCSegment and UploadedSegment names so we can save those ZK accesses. When merging segments, we can take max(seqIds of input segments) and encode it in the new segment's name via the UploadedSegment name pattern. This way LLCSegment and UploadedSegment seqId can match too. So we might have to put a fallback logic of using segment creation time but that can lead to inconsistencies often. -- 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
