Re: [PR] Add prefetching for terms dict in doc values [lucene]

2025-06-18 Thread via GitHub
easyice commented on PR #14773: URL: https://github.com/apache/lucene/pull/14773#issuecomment-2984815277 Thanks for the explanation, You're right, I'll close this PR. I feel an API to prefetch full range of data could be useful, e.g. for files like `.tip`, which are relatively small but hav

Re: [PR] Add prefetching for terms dict in doc values [lucene]

2025-06-18 Thread via GitHub
easyice closed pull request #14773: Add prefetching for terms dict in doc values URL: https://github.com/apache/lucene/pull/14773 -- 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.

Re: [PR] Add prefetching for terms dict in doc values [lucene]

2025-06-18 Thread via GitHub
jpountz commented on PR #14773: URL: https://github.com/apache/lucene/pull/14773#issuecomment-2983993101 Doing this is a bit less obvious in my mind since terms dictionaris are allowed to have a random access pattern, when doc-value iterators are required to be consumed in order? Som

[PR] Add prefetching for terms dict in doc values [lucene]

2025-06-12 Thread via GitHub
easyice opened a new pull request, #14773: URL: https://github.com/apache/lucene/pull/14773 This follows a similar approach as doc values and only prefetches the first page of data. Perhaps these were missed at the time? -- This is an automated message from the Apache Git Service. To res

Re: [PR] Add prefetching for terms dict in doc values [lucene]

2025-06-12 Thread via GitHub
github-actions[bot] commented on PR #14773: URL: https://github.com/apache/lucene/pull/14773#issuecomment-2966493936 This PR does not have an entry in lucene/CHANGES.txt. Consider adding one. If the PR doesn't need a changelog entry, then add the skip-changelog label to it and you will stop