[jira] Commented: (LUCENE-2149) Simplify Spatial LatLng and LLRect classes

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (LUCENE-2149) Simplify Spatial LatLng and LLRect classes

JIRA jira@apache.org

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

Simon Willnauer commented on LUCENE-2149:
-----------------------------------------

Are we still working on this issue? It seems to be somewhat outdated though and I am not sure about the geospatial status in solr - any ideas? If we do not work on this I would like to close as won't fix.

> Simplify Spatial LatLng and LLRect classes
> ------------------------------------------
>
>                 Key: LUCENE-2149
>                 URL: https://issues.apache.org/jira/browse/LUCENE-2149
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: contrib/spatial
>    Affects Versions: 3.1
>            Reporter: Chris Male
>            Assignee: Simon Willnauer
>         Attachments: LUCENE-2149.patch
>
>
> Currently in the contrib there is FloatLatLng, and FixedLatLng, which both extend LatLng.  The reason for this separation is not clear and is not needed in the current functionality.  The functionality that is used can be collapsed into LatLng, which can be made a concrete class.  Internally LatLng can benefit from the improvements suggested in LUCENE-1934.
> LLRect, which uses LatLng, can also be simplified by removing the unused functionality, and using the new LatLng class.
> All classes can be improved through documentation, some method renaming, and general code tidy up.

--
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: [hidden email]
For additional commands, e-mail: [hidden email]