alessandrobenedetti commented on PR #2489:
URL: https://github.com/apache/solr/pull/2489#issuecomment-2174445957

   > I did the work of migrating to a QParser but don't have permissions to 
push to your PR branch.
   > 
   > Beware, I'm reverting most/all of the files that you modified of existing 
Solr source files. If you would prefer I submit a separate PR, I'll do that. 
Instead of `combiner=true`, you'd do `q={!combine}`.
   I don't remember if this is the best way to invite you as a collaborator but 
I just added you on our repo, hopefully that works!
   
   I wasn't entirely sure a Query Parser is the right place for this, given we 
are not really building a "new query" but then I thought : "hey we have the 
boolean query parser, that more or less does the same", so I have nothing 
against it!
   
   You can go ahead and add your commit, I don't care much if you overwrote 
many of my changes, I just want a nice and clean solution, however we come up 
to it! Worst case scenario we go back in commits and re-use what's necessary!
   > 
   > Wouldn't the docSet (assuming it's needed, like for faceting) be computed 
on each subQuery, and if so how is that rolled up to the final QueryResult? I 
see you tested that faceting works but I suspect there may be a very sad 
performance bug here.
   
   
   
   In regards to the docSet It was rolled up in:
   `SortedIntDocSet docSet = new SortedIntDocSet(combinedResultsDocIds, 
combinedResultsLength);
       combinedRankedList.setDocSet(docSet);`
       
   combinedResultsDocIds is the merged array of doc Ids.
   I admit I haven't spent the required time on this part and it was quickly 
done as a draft, so feel free to recommend a different way/correct me!
       


-- 
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...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to