jpountz commented on code in PR #14294:
URL: https://github.com/apache/lucene/pull/14294#discussion_r1971606728


##########
lucene/core/src/java/org/apache/lucene/search/ScorerUtil.java:
##########
@@ -113,22 +116,21 @@ static Scorable likelyTermScorer(Scorable scorable) {
 
   /**
    * Optimize {@link Bits} representing the set of accepted documents for the 
case when it is likely
-   * implemented via a {@link FixedBitSet}. This helps make calls to {@link 
Bits#get(int)}
-   * inlinable, which in-turn helps speed up query evaluation. This is 
especially helpful as
-   * inlining will sometimes enable auto-vectorizing shifts and masks that are 
done in {@link
-   * FixedBitSet#get(int)}.
+   * implemented as live docs. This helps make calls to {@link Bits#get(int)} 
inlinable, which
+   * in-turn helps speed up query evaluation. This is especially helpful as 
inlining will sometimes
+   * enable auto-vectorizing shifts and masks that are done in {@link 
FixedBitSet#get(int)}.
    */
-  static Bits likelyFixedBitSet(Bits acceptDocs) {
-    if (acceptDocs instanceof FixedBitSet) {
+  static Bits likelyLiveDocs(Bits acceptDocs) {
+    if (acceptDocs == null) {
+      return acceptDocs;
+    } else if (acceptDocs.getClass() == DEFAULT_ACCEPT_DOCS_CLASS) {
       return acceptDocs;
-    } else if (acceptDocs != null) {
-      return new FilterBits(acceptDocs);
     } else {
-      return null;
+      return new FilterBits(acceptDocs);
     }
   }
 
-  private static class FilterBits implements Bits {
+  static class FilterBits implements Bits {

Review Comment:
   This is because I needed it in the test, which I had failed to check in. I 
have now checked it in, but your comment made me want to avoid making this 
class pkg-private, so I refactored the test. This class is now private again.



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