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

Nicholas Knize commented on LUCENE-6450:
----------------------------------------

bq. It's only loosely coupled to the spatial module right now.

I've seen the code. Its well designed.  My point is that its in the spatial 
module right now. It would need to be taken out of the spatial module for it to 
be usable in core and by this patch. Sounds like a separate issue for lucene 
committer buy in.

bq. Most GIS code can readily map a point on an earth model to a point on a 
sphere.

I'm not arguing with you. My point refers to the, maybe 2%, class of lucene geo 
users you mentioned. Those same Cartographers that would counter-argue that 
geo3d is not ISO 19107 compliant so sphere accuracy is not good enough for them 
(enjoy those passionate discussions at FOSS4G). Don't misunderstand me - I'm 
not calling your baby ugly. There's great stuff there and I'm all for 
investigating using it to improve the simple approach in this patch.  In the 
meantime, since it sounds like accuracy concerns for the large poly use case 
are for real?, it might be easier just to add the few lines of mercator 
re-projection code to the post-filter and investigate using geo3d in a phase 2 
performance improvement (if it becomes available to core).

> Add simple encoded GeoPointField type to core
> ---------------------------------------------
>
>                 Key: LUCENE-6450
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6450
>             Project: Lucene - Core
>          Issue Type: New Feature
>    Affects Versions: Trunk, 5.x
>            Reporter: Nicholas Knize
>            Priority: Minor
>         Attachments: LUCENE-6450-5x.patch, LUCENE-6450-TRUNK.patch, 
> LUCENE-6450.patch, LUCENE-6450.patch, LUCENE-6450.patch, LUCENE-6450.patch
>
>
> At the moment all spatial capabilities, including basic point based indexing 
> and querying, require the lucene-spatial module. The spatial module, designed 
> to handle all things geo, requires dependency overhead (s4j, jts) to provide 
> spatial rigor for even the most simplistic spatial search use-cases (e.g., 
> lat/lon bounding box, point in poly, distance search). This feature trims the 
> overhead by adding a new GeoPointField type to core along with 
> GeoBoundingBoxQuery and GeoPolygonQuery classes to the .search package. This 
> field is intended as a straightforward lightweight type for the most basic 
> geo point use-cases without the overhead. 
> The field uses simple bit twiddling operations (currently morton hashing) to 
> encode lat/lon into a single long term.  The queries leverage simple 
> multi-phase filtering that starts by leveraging NumericRangeQuery to reduce 
> candidate terms deferring the more expensive mathematics to the smaller 
> candidate sets.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to