Build Upgraded to Gradle 8.4

2023-10-11 Thread Kevin Risden
Just pushed to main/branch_9x Gradle 8.4 upgrade. Let me know if you see any weirdness. Google Java Format was upgraded so there is a tidy commit that made a bunch of whitespace changes. If you have PRs in progress, probably need to make sure tidy is run after getting up to date with

Re: Weird HNSW merge performance result

2023-10-11 Thread Patrick Zhai
Hi Ben, Thanks! I think that's the issue! I was using some old local checkout. I will try with the latest commit and report back if the results still look weird. On Wed, Oct 11, 2023, 12:26 Benjamin Trent wrote: > Heya Patrick, > > What version of Lucene Util are you using? There was a bug

Re: Weird HNSW merge performance result

2023-10-11 Thread Benjamin Trent
Heya Patrick, What version of Lucene Util are you using? There was a bug where `forceMerge` was not actually using your configured maxConn & beamWidth. See: https://github.com/mikemccand/luceneutil/pull/232 Do you have that commit and rebuilt the KnnGraphTester? On Wed, Oct 11, 2023 at 10:10 AM

Re: Weird HNSW merge performance result

2023-10-11 Thread Patrick Zhai
Hi Adrien, I'm using the default CMS, but I doubt whether the merge will be triggered at all in the background. Since no merge policy is changed the default TMP will likely only merge the segments after they reach 10 I believe? But the index is about 300M and the buffer size is around 50M so I