Mailing-List: contact dev-help@lucene.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@lucene.apache.org
Date: Thu, 28 Jun 2012 05:40:44 +0000 (UTC)
From: "David Smiley (JIRA)" <jira@apache.org>
To: dev@lucene.apache.org
Message-ID: <249279220.65571.1340862044213.JavaMail.jiratomcat@issues-vm>
In-Reply-To: <1757892889.61407.1340796523784.JavaMail.jiratomcat@issues-vm>
Subject: [jira] [Commented] (LUCENE-4167) Remove the use of SpatialOperation
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

David Smiley commented on LUCENE-4167:
--------------------------------------

I looked at the patch.  {{shrug}}  Some things are marginally simpler but it seems like a step back for supporting anything other than one spatial operation.  Please hold off for a few days before committing without more consensus; I'd like to hear Ryan's point of view on this issue as it's a big deal.
                
> Remove the use of SpatialOperation
> ----------------------------------
>
>                 Key: LUCENE-4167
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4167
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: modules/spatial
>            Reporter: Chris Male
>         Attachments: LUCENE-4167.patch
>
>
> Looking at the code in TwoDoublesStrategy I noticed SpatialOperations.BBoxWithin vs isWithin which confused me.  Looking over the other Strategys I see that really only isWithin and Intersects is supported.  Only TwoDoublesStrategy supports IsDisjointTo.  The remainder of SpatialOperations are not supported.
> I don't think we should use SpatialOperation as this stage since it is not clear what Operations are supported by what Strategys, many Operations are not supported, and the code for handling the Operations is usually the same.  We can spin off the code for TwoDoublesStrategy's IsDisjointTo support into a different Strategy.

--
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: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org