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

Simon Willnauer commented on LUCENE-2139:
-----------------------------------------

Thank you Chris!
Lets go with small issues from here. I suggest we start with a svn move / copy 
issue that bring names and classes in the right place. With the size and the 
number of changes in patch this make the most sense to me. The good thing with 
spatial is that we can change class names and locations / packages as all the 
API is marked experimental. Once we have this in place people can easily apply 
the patch and join the issues. 
If this is done I suggest to start from top to bottom package wise which will 
guarantee that we have a solid package strucuture and will make patches and 
dependency handling way easier. 
I want to drive towards a solid contrib module (you code is a huge step 
towards) so making small steps will eventually bring us that way. IMO it will 
also speed things up as it is easier to review and to commit. 

Thoughts?

> Cleanup and Improvement of Spatial Contrib
> ------------------------------------------
>
>                 Key: LUCENE-2139
>                 URL: https://issues.apache.org/jira/browse/LUCENE-2139
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: contrib/spatial
>    Affects Versions: 3.1
>            Reporter: Chris Male
>            Assignee: Simon Willnauer
>         Attachments: LUCENE-2139-Java5Only.patch, LUCENE-2139.patch
>
>
> The current spatial contrib can be improved by adding documentation, tests, 
> removing unused classes and code, repackaging the classes and improving the 
> performance of the distance filtering.  The latter will incorporate the 
> multi-threaded functionality introduced in LUCENE-1732.  
> Other improvements involve adding better support for different distance 
> units, different distance calculators and different data formats (whether it 
> be lat/long fields, geohashes, or something else in the future).
> Patch to be added soon.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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

Reply via email to