vigyasharma commented on issue #14362: URL: https://github.com/apache/lucene/issues/14362#issuecomment-2752804917
Thanks @guojialiang92 . Is the plan here to support creating an IndexWriter with a supplied value of `counter`, say `N`, so that all it's commit generations are `>=N` i.e. `segments_N, segments_N+1, ...` and so on ? To confirm my understanding, when a primary dies, one cannot really guarantee that all replicas were fully caught up. If the winning replica (new primary) was lagging behind, and you simply continue to use its `SegmentInfos#counter`, it might end up overwriting some segment files in other replicas. So you use raft to select the right counter value and start segments from that value. This is specifically a problem for segment replication. Regular document replication works fine because each document is reindexed anyway and segment files are not copied over. Is that more or less correct? Anyway, I don't see any problems with this support and it does have a valid use-case. If you want to raise a PR, I can help review it. -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
