benwtrent commented on PR #13463: URL: https://github.com/apache/lucene/pull/13463#issuecomment-2155506996
OK, maybe there is a bigger bug here, or this bug made this even better and we can reduce some constants to improve performance. I benchmarked over 1M 768 vectors, flushing every 12MB to get some segments. This resulted in 25 segments (some fairly small). With your change: ``` recall latency nDoc fanout maxConn beamWidth visited 0.909 12.03 1000000 0 16 100 31658 ``` Baseline: ``` recall latency nDoc fanout maxConn beamWidth visited 0.875 5.37 1000000 0 16 100 13222 ``` The patch for your change: [patch.patch](https://github.com/user-attachments/files/15744684/patch.patch) -- 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
