gsmiller commented on PR #12089:
URL: https://github.com/apache/lucene/pull/12089#issuecomment-1419344879

   > but there is also one benchmark where it does substantially worse ("Low 
Cardinality + High Cost Country Code Filter Terms").
   
   100%. The issue here is that `TermInSetQuery` gets rewritten to a 
`BooleanQuery` because there are fewer than 16 terms, so it doesn't have a 
chance to "self-optimize" to use doc values. We can fix this by not eagerly 
rewriting to a `BooleanQuery`, but I held off doing that for now. So this is 
"easily" fixable I think.


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