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

David Smiley commented on SOLR-5163:
------------------------------------

Looks pretty good.  Though I don't think you tested this change?  Also, if 
there's an infinite loop in aliasing, what will happen?  I believe this 
resulted in a failure before but I wonder if now, due to this new code, Solr 
will loop forever?  Perhaps you should in one shot both do the recursive 
aliasing detection (moving from where is now) and also validate the field?

> edismax should throw exception when qf refers to nonexistent field
> ------------------------------------------------------------------
>
>                 Key: SOLR-5163
>                 URL: https://issues.apache.org/jira/browse/SOLR-5163
>             Project: Solr
>          Issue Type: Bug
>          Components: query parsers, search
>    Affects Versions: 4.10.4
>            Reporter: Steven Bower
>            Assignee: David Smiley
>            Priority: Major
>              Labels: newdev
>         Attachments: SOLR-5163.patch, SOLR-5163.patch, SOLR-5163.patch
>
>
> query:
> q=foo AND bar
> qf=field1
> qf=field2
> defType=edismax
> Where field1 exists and field2 doesn't..
> will treat the AND as a term vs and operator



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