[
https://issues.apache.org/jira/browse/LUCENE-8369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16902133#comment-16902133
]
Simon Willnauer commented on LUCENE-8369:
-----------------------------------------
I don't think we should scarify the existence of LatLong point searching out of
core for the sake of code visibility. I think we should keep it in core and
open up visibility to enable code-reuse in the modules and use
[email protected]_ in order to mark classes as internal and prevent users from
complaining when the API changes. It's not ideal but progress. Can we separate
the disucssion of getting rid of the spacial module from graduating the various
shapes from sandbox to wherever? I think keeping a module for 2 classes doesn't
make sense. We can move those two classes to core too or even get rid of them
altogether I don't think it should influence the discussion if something else
should be graduated.
One other option would be we move all non-core spacials from sandbox to spatial
as long as they don't add any additional dependency. that would be an
intermediate step. we can still graduate from there then.
> Remove the spatial module as it is obsolete
> -------------------------------------------
>
> Key: LUCENE-8369
> URL: https://issues.apache.org/jira/browse/LUCENE-8369
> Project: Lucene - Core
> Issue Type: Task
> Components: modules/spatial
> Reporter: David Smiley
> Assignee: David Smiley
> Priority: Major
> Attachments: LUCENE-8369.patch
>
>
> The "spatial" module is at this juncture nearly empty with only a couple
> utilities that aren't used by anything in the entire codebase --
> GeoRelationUtils, and MortonEncoder. Perhaps it should have been removed
> earlier in LUCENE-7664 which was the removal of GeoPointField which was
> essentially why the module existed. Better late than never.
--
This message was sent by Atlassian JIRA
(v7.6.14#76016)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
