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

Reply via email to