[
https://issues.apache.org/jira/browse/LUCENE-7951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16158617#comment-16158617
]
David Smiley commented on LUCENE-7951:
--------------------------------------
bq. We must always construct the calculator if initCalculator() is called as
the planet model can be different.
Oh I see. Instead I suggest letting the planet model and calculator fields be
null initially. Overriding factory.newSpatialContext will allow us to do these
checks. This way, init(map) is still optional -- consistent with the other
SpatialContextFactory impls. There is a similar situation for "worldBounds".
In that case, the default SpatialContext impl knows checks and has defaulting
logic Since we don't have a custom SpatialContext impl for Geo3D, we could
just as easily override factory.newSpatialContext.
The PR looks good; I'll make further comments there. Thanks for doing a PR.
> New wrapper classes for Geo3d
> -----------------------------
>
> Key: LUCENE-7951
> URL: https://issues.apache.org/jira/browse/LUCENE-7951
> Project: Lucene - Core
> Issue Type: Improvement
> Components: modules/spatial-extras
> Reporter: Ignacio Vera
> Assignee: David Smiley
> Priority: Minor
> Attachments: LUCENE_7951_build.patch, LUCENE_7951_build.patch,
> LUCENE-7951.patch, LUCENE-7951.patch
>
>
> Hi,
> After the latest developments in the Geo3d library, in particular:
> [https://issues.apache.org/jira/browse/LUCENE-7906] : Spatial relationships
> between GeoShapes
> [https://issues.apache.org/jira/browse/LUCENE-7936]: Serialization of
> GeoShapes.
> I propose a new set of wrapper classes which can be for example linked to
> Solr as they implement their own SpatialContextFactory. It provides the
> capability of indexing shapes with
> spherical geometry.
> Thanks!
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
