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

David Smiley edited comment on LUCENE-4173 at 7/26/12 3:34 PM:
---------------------------------------------------------------

I was thinking this, yeah... and even though I wouldn't use it and would not 
support it being a default, InvalidShape.IGNORE -- haha full circle on the 
issue but I think the enum solves it.  Well maybe not completely because it 
makes no sense to ignore a query shape, so what would it mean to set this to 
IGNORE and the query shape is incompatible?  So maybe no IGNORE?  So default 
FAIL is your preference?
                
      was (Author: dsmiley):
    I was thinking this, yeah... and even though I wouldn't use it and would 
not support it being a default, InvalidShape.IGNORE -- haha full circle on the 
issue but I think the enum solves it.  So default FAIL is your preference?
                  
> Remove IgnoreIncompatibleGeometry for SpatialStrategys
> ------------------------------------------------------
>
>                 Key: LUCENE-4173
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4173
>             Project: Lucene - Core
>          Issue Type: Bug
>          Components: modules/spatial
>            Reporter: Chris Male
>            Assignee: David Smiley
>         Attachments: LUCENE-4173.patch
>
>
> Silently not indexing anything for a Shape is not okay.  Users should get an 
> Exception and then they can decide how to proceed.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to