DivyanshIITB commented on issue #14348: URL: https://github.com/apache/lucene/issues/14348#issuecomment-2717999626
Hi @viliam-durina, I find this issue interesting and would like to work on it. I see that Lucene currently uses ReadAdvice.RANDOM when opening vector files (.vec and .veq). Switching to RANDOM_PRELOAD seems reasonable, especially if it helps reduce performance degradation when the files don’t fit in the page cache. To proceed, I plan to: Identify where ReadAdvice.RANDOM is used in Lucene’s codebase for opening vector files. Modify it to ReadAdvice.RANDOM_PRELOAD. Benchmark and test the performance impact to confirm the improvement. Run the test suite to ensure no regressions. Before making the change, I’d love to hear your thoughts: Have you already tested the impact of RANDOM_PRELOAD, and does it consistently show improvements in performance? Would you recommend making this change across all cases where ReadAdvice.RANDOM is used, or only for vector file access? Looking forward to your feedback! If this is good to go, I’d be happy to submit a PR. Thanks! -- 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
