Re: [PR] Clean up variable-gaps terms format. [lucene]

2024-03-27 Thread via GitHub
jpountz merged PR #13216: URL: https://github.com/apache/lucene/pull/13216 -- 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.apa

Re: [PR] Clean up variable-gaps terms format. [lucene]

2024-03-27 Thread via GitHub
jpountz commented on PR #13216: URL: https://github.com/apache/lucene/pull/13216#issuecomment-2023348250 I'm merging only to `main` for now. -- 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 sp

Re: [PR] Clean up variable-gaps terms format. [lucene]

2024-03-26 Thread via GitHub
uschindler commented on PR #13216: URL: https://github.com/apache/lucene/pull/13216#issuecomment-2020751491 > > we now use ChecksumIndexInput as the file is fully readonce without seeking. Because we removed the IOContext from directory's open method (as it is hardcoded to readOnce), we do

Re: [PR] Clean up variable-gaps terms format. [lucene]

2024-03-26 Thread via GitHub
jpountz commented on PR #13216: URL: https://github.com/apache/lucene/pull/13216#issuecomment-2020509119 > we now use ChecksumIndexInput as the file is fully readonce without seeking. Because we removed the IOContext from directory's open method (as it is hardcoded to readOnce), we do not n

[PR] Clean up variable-gaps terms format. [lucene]

2024-03-26 Thread via GitHub
jpountz opened a new pull request, #13216: URL: https://github.com/apache/lucene/pull/13216 The variable-gaps terms format uses the legacy storage layout of storing metadata at the end of the index file, and storing the start pointer of the metadata as the last 8 bytes of the index files (j