uschindler commented on PR #15341: URL: https://github.com/apache/lucene/pull/15341#issuecomment-3713792361
Hi, I think we can start with aligning the vector files itsself. For CFS files: I'd like to keep the default alignment for now, but maybe we can add a modification in the new CompundFileWriter that it just uses another alignment when the filename ends in a vector file. I know this is not nice and a more generic way would be fine, but as we currently just copy the data unmodified without any codec support to a CFS file, the only way to control this is by file name extension. We can do this in a separate PR, but if you want to do a simple change in the CompoundFileWriter of the actual codec to se 64 alignment, just do it. As file format doe snot change, we don't need a new codec version. I think a method returning alignment based on filename would be fine. Uwe -- 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]
