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

    https://github.com/apache/lucene-solr/pull/162#discussion_r185490261
  
    --- Diff: 
solr/core/src/java/org/apache/solr/handler/component/QueryComponent.java ---
    @@ -1414,10 +1267,17 @@ private void 
doProcessGroupedDistributedSearchFirstPhase(ResponseBuilder rb, Que
             .setSearcher(searcher);
     
         for (String field : groupingSpec.getFields()) {
    +      final int topNGroups;
    +      Query query = cmd.getQuery();
    +      if (query instanceof AbstractReRankQuery){
    +        topNGroups = cmd.getOffset() + 
((AbstractReRankQuery)query).getReRankDocs();
    +      } else {
    +        topNGroups = cmd.getOffset() + cmd.getLen();
    +      }
           topsGroupsActionBuilder.addCommandField(new 
SearchGroupsFieldCommand.Builder()
               .setField(schema.getField(field))
               .setGroupSort(groupingSpec.getGroupSort())
    -          .setTopNGroups(cmd.getOffset() + cmd.getLen())
    --- End diff --
    
    True, this currently reranks the top `start +  rerankDocuments `, I'll fix 
it. Thank you


---

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

Reply via email to