Github user salsolatragus commented on a diff in the pull request:

    https://github.com/apache/lucene-solr/pull/446#discussion_r215380343
  
    --- Diff: lucene/core/src/java/org/apache/lucene/search/BooleanWeight.java 
---
    @@ -40,28 +39,40 @@
       /** The Similarity implementation. */
       final Similarity similarity;
       final BooleanQuery query;
    -  
    -  final ArrayList<Weight> weights;
    +
    +  private static class WeightedBooleanClause {
    +    BooleanClause clause;
    --- End diff --
    
    The code also uses some convenience methods on `BooleanClause` 
(`isRequired()`, `isScoring()`, and `isProhibited()`). Those are only checking 
the `BooleanCause#Occur`, but these checks are used elsewhere, so we should not 
inline them, I suppose. Therefore, we need the entire clause.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to