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

ASF subversion and git services commented on SOLR-12249:
--------------------------------------------------------

Commit 48b026d5eedcfa2090596639dd5282fac18db40a in lucene-solr's branch 
refs/heads/master from Munendra S N
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=48b026d ]

SOLR-13404: Fix NPE when group=true and no group.field is present

* This was introduced in SOLR-12249


> Better error message when grouping on a tokenized (non SortableText) field in 
> SolrCloud
> ---------------------------------------------------------------------------------------
>
>                 Key: SOLR-12249
>                 URL: https://issues.apache.org/jira/browse/SOLR-12249
>             Project: Solr
>          Issue Type: Bug
>    Affects Versions: 6.6.2, 8.0
>            Reporter: Erick Erickson
>            Assignee: Erick Erickson
>            Priority: Minor
>             Fix For: master (9.0), 8.2
>
>         Attachments: SOLR-12249.patch
>
>
> I didn't test this with master. Under the covers in stand-alone mode, the 
> "min" function is silently  substituted for the grouping, but that's not true 
> in SorlCloud mode. I broke this JIRA out separately to discuss whether it 
> _ever_ makes sense to group by a tokenized text field.
> Grouping by the min value in a field is at least consistent, but on a text 
> field I don't think it makes sense.
> I propose that we explicitly dis-allow this in both stand-alone and Cloud 
> mode, especially now that there's the SortableTextField.
> Comments?



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to