martijnvg commented on issue #14881: URL: https://github.com/apache/lucene/issues/14881#issuecomment-3022712602
> we must fully check on merge: I fully agree that on merge a full check is required. I'm wondering whether we can avoid doing a full check at flush time for the tmp files that get created for stored fields when index sorting is enabled. From my understanding `SortingStoredFieldsConsumer` is only used when flushing a new segment. At merge time, there is no need to create a tmp file for stored fields. I noticed that in other formats, when creating a tmp file not always a full check is performed. With that in mind, I wonder whether we can omit the full integrity check that `SortingStoredFieldsConsumer` is doing. -- 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
