Skip to content

SOLR-17319: CompoundQueryComponent scribbles#2597

Draft
cpoerschke wants to merge 5 commits intoapache:mainfrom
cpoerschke:SOLR-17319-compound-query-component
Draft

SOLR-17319: CompoundQueryComponent scribbles#2597
cpoerschke wants to merge 5 commits intoapache:mainfrom
cpoerschke:SOLR-17319-compound-query-component

Conversation

@cpoerschke
Copy link
Copy Markdown
Contributor


public class CompoundSearchHandler extends SearchHandler {

protected ResponseBuilder newResponseBuilder(
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://issues.apache.org/jira/browse/SOLR-13511 added support to allow customisation like this.

Comment on lines +29 to +35
ResponseBuilder rb_1 = new ResponseBuilder(rb.req, new SolrQueryResponse(), rb.components);
rb_1.setQueryString(rb_1.req.getParams().get("rrf.q.1"));
crb.responseBuilders.add(rb_1);

ResponseBuilder rb_2 = new ResponseBuilder(rb.req, new SolrQueryResponse(), rb.components);
rb_2.setQueryString(rb_2.req.getParams().get("rrf.q.2"));
crb.responseBuilders.add(rb_2);
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QueryComponent.prepare already has support for using other than CommonParams.Q i.e. q parameter for the query: https://github.com/apache/solr/blob/releases/solr/9.6.1/solr/core/src/java/org/apache/solr/handler/component/QueryComponent.java#L174-L181

@cpoerschke
Copy link
Copy Markdown
Contributor Author

#3648 is a continuation of these scribbles here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant