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

   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}`.
   
   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.


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