jpountz commented on PR #14203: URL: https://github.com/apache/lucene/pull/14203#issuecomment-2726996663
Again, thanks a lot for running benchmarks. > I can refactor the code to the specialized decoding if it makes sense to you That would be great, thank you. Sorry for making it hard for you to move this PR forward, I was a bit annoyed that we needed something complicated to speed things up, I like the simplicity of specializedDecodeMaskInRemainder. > BTW, should we disable the flexibility of changing maxPointsInLeafNode if we make optimization only work on default config? In my opinion, it's good enough the way things are today as the default codec doesn't allow configuring the number of points per leaf node. -- 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