[SOLR-6202] Simplify the use of "rq" for query re-ranking

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Assignee: Unassigned
• Reporter: Chris M. Hostetter
• Created: 25/Jun/14 18:30
• Updated: 08/Oct/19 14:50

Description

It seems like there is some really easy tweaks we could make to the "rq" (Re-Ranking Query) API to make it less verbose and easier to use:

• in QueryComponent
  • when parsing "rq", we should use "rerank" as the default parser
    • should be a trivial change to the QParser.getParser when initializin rqparser
• in ReRankQParserPlugin
  • we should deprecate/remove the "reRankQuery" local param and just use "v"
    • this is the implicit param created when using the body of the query string, or it can be explicit (to refer to variables)
  • we should leverage subQuery() (just like NestedQParser does) so that "defType" is respected when parsing the wrapped query

Unless i'm overlooking something, the combination of these simple changes should drastically simplify a lot of use cases...

BEFORE: rq={!rerank reRankQuery=$rqq}rqq=XXX
 AFTER: rq=XXX

BEFORE: rq={!rerank reRankQuery=$rqq reRankDocs=200}$rqq={!func}XXX
 AFTER: rq={!reRankDocs=200 defType=func}XXX

Issue Links

• is related to: SOLR-6835 ReRankQuery throws NullPointerException