uschindler commented on PR #15341: URL: https://github.com/apache/lucene/pull/15341#issuecomment-3716705320
> Thanks for the input @uschindler, I added an extension-based function for now -- although I'd prefer a longer term solution for file-specific alignment (or simply using the LCM of all byte alignments) -- but we can take that up later I think the current solution is not too bad. It is still part of default codec, so hardcoding filename patterns is ok. I agree, we should use the LCM as alignment but we can compensate it's effect by writing the files in a different order. We have all file sizes, so we could do some defragmenting approach to order the files so we waste less bytes. But I don't think that 64 bytes is really a huge overhead. Come on an index with several gigabytes and 20 files per segment only wastes less than a kilobyte per segment. 😜 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
