jpountz opened a new pull request, #14033:
URL: https://github.com/apache/lucene/pull/14033

   Recent speedups by making call sites bimorphic made me want to play with 
combining all postings enums and impacts enums of the default codec into a 
single class, in order to reduce polymorphism. Unfortunately, it does not yield 
a speedup since the major polymorphic call sites we have that hurt performance 
(`DefaultBulkScorer`, `ConjunctionDISI`) are still 3-polymorphic or more.
   
   Yet, reduced polymorphism at little performance impact is a good trade-off 
as it would help make call sites bimorphic for users who don't have as much 
query diversity as nightly benchmarks, or in the future when we remove other 
causes of polymorphism.


-- 
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