[
https://issues.apache.org/jira/browse/LUCENE-7056?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15172249#comment-15172249
]
David Smiley commented on LUCENE-7056:
--------------------------------------
Specifics:
* Rename module "spatial3d" to "spatial-geom3d" or perhaps "spatial-3d-geom" in
order to emphasizes the geometry aspect
* Move package “org.apache.lucene.geo3d” to “org.apache.lucene.spatial.3d.geom”.
* Move Geo3DPoint and PointInGeo3DShapeQuery to the lucene-spatial-extras
module to the package “org.apache.lucene.spatial.3d”
** Remove dependency from lucene-spatial-geom3d to Lucene. Ensure that the new
module’s Maven POM has no runtime dependencies at all.
* Move Geo3dShape in “org.apache.lucene.spatial.spatial4j” to
“org.apache.lucene.spatial.3d” to keep the 3d related stuff here together with
the 2 other classes moving over.
Note “Geo3D” will then cease to be a package/brand in here. It’s basically our
spatial 3d geometry module.
> Spatial3d/Geo3d should have zero runtime dependencies
> -----------------------------------------------------
>
> Key: LUCENE-7056
> URL: https://issues.apache.org/jira/browse/LUCENE-7056
> Project: Lucene - Core
> Issue Type: Improvement
> Components: modules/spatial3d
> Reporter: David Smiley
> Assignee: David Smiley
> Fix For: 6.0
>
>
> This is a proposal for the "spatial3d" module to be purely about the
> shape/geometry implementations it has. In Lucene 5 that's actually all it
> has. In Lucene 6 at the moment its ~76 files have 2 classes that I think
> should go elsewhere: Geo3DPoint and PointInGeo3DShapeQuery. Specifically
> lucene-spatial-extras (which doesn't quite exist yet so lucene-spatial) would
> be a suitable place due to the dependency. _Eventually_ I see this module
> migrating elsewhere be it on its own or a part of something else more
> spatial-ish. Even if that never comes to pass, non-Lucene users who want to
> use this module for it's geometry annoyingly have to exclude the Lucene
> dependencies that are there because this module also contains these two
> classes.
> In a comment I'll suggest some specifics.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
