lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Smiley (JIRA)" <>
Subject [jira] [Resolved] (LUCENE-4188) Storing Shapes shouldn't be Strategy dependent
Date Wed, 11 Jul 2012 06:33:34 GMT


David Smiley resolved LUCENE-4188.

       Resolution: Fixed
    Fix Version/s: 4.0

I inlined/removed SpatialStrategy.createStoredField() and added documentation to the SpatialStrategy
class header & to createFields() regarding storing a field value.


I'll address the rename of createFields() to createIndexableFields() in LUCENE-4192
> Storing Shapes shouldn't be Strategy dependent
> ----------------------------------------------
>                 Key: LUCENE-4188
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: modules/spatial
>            Reporter: Chris Male
>            Assignee: David Smiley
>             Fix For: 4.0
>         Attachments: LUCENE-4188_remove_field_storage_from_createField.patch
> The logic for storing Shape representations seems to be different for each Strategy.
 The PrefixTreeStrategy impls store the Shape in WKT, which is nice if you're using WKT but
not much help if you're not.  BBoxStrategy doesn't actually store the Shape itself, but a
representation of the bounding box.  TwoDoubles seems to follow the PrefixTreeStrategy approach,
which is surprising since it only indexes Points and they could be stored without using WKT.
> I think we need to consider what storing a Shape means.  If we want to store the Shape
itself, then that logic should be standardised and done outside of the Strategys since it
is not really related to them.  If we want to store the terms being used by the Strategys
to make Shapes queryable, then we need to change the logic in the Strategys to actually do

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message