[ 
https://issues.apache.org/jira/browse/SOLR-6234?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14586444#comment-14586444
 ] 

Ryan Josal commented on SOLR-6234:
----------------------------------

I've been using this for a few days now, and here are my comments:
*) it's more functionally useful to have the scores returned with this than the 
!join parser.
*) basing it off JoinUtil is a clean solution that centralizes this type of 
logic.
*) That being said, scorejoin has performed worse than !join in my tests, 
usually almost twice as slow.  It's not an exhaustive test by any means, just 
an observation.  So it doesn't seem fit to replace !join as it is, although it 
does feel like this is the right direction to be going for joins.
*) JoinUtil does not currently support multivalue fields for the "to" index.  
It chooses one of the matching values instead of applying the ScoreMode for all 
matching documents.  It's a fairly small change to TermsIncludingScoreQuery to 
add this feature.

> Scoring modes for query time join 
> ----------------------------------
>
>                 Key: SOLR-6234
>                 URL: https://issues.apache.org/jira/browse/SOLR-6234
>             Project: Solr
>          Issue Type: New Feature
>          Components: query parsers
>    Affects Versions: 5.3
>            Reporter: Mikhail Khludnev
>              Labels: features, patch, test
>             Fix For: 5.3
>
>         Attachments: SOLR-6234.patch, SOLR-6234.patch
>
>
> it adds {{scorejoin}} query parser which calls Lucene's JoinUtil underneath. 
> It supports:
> - {{score=none|avg|max|total}} local param (passed as ScoreMode to JoinUtil)
>  - {{score=none}} is *default*, eg if you *omit* this localparam 
> - supports {{b=100}} param to pass {{Query.setBoost()}}.
> - {{multiVals=true|false}} is introduced 
> - there is a test coverage for cross core join case. 
> - so far it joins string and multivalue string fields (Sorted, SortedSet, 
> Binary), but not Numerics DVs. follow-up LUCENE-5868  
> -there was a bug in cross core join, however there is a workaround for it- 
> it's fixed in Dec'14 patch.
> Note: the development of this patch was sponsored by an anonymous contributor 
> and approved for release under Apache License.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to