Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Solr config refers to deprecated LatLonType #1370

Closed
jcoyne opened this issue Nov 1, 2018 · 1 comment
Closed

Solr config refers to deprecated LatLonType #1370

jcoyne opened this issue Nov 1, 2018 · 1 comment

Comments

@jcoyne
Copy link
Member

jcoyne commented Nov 1, 2018 •
edited 

    <fieldType name="location" class="solr.LatLonType" subFieldSuffix="_coordinate"/>

I don't think we're using this anywhere. Can we drop this field type or do we want to migrate to one of: LatLonPointSpatialField, SpatialRecursivePrefixTreeField, or RptWithGeometrySpatialField?
@no-reply
Copy link
Member

Shall we just remove it? It seems like it won't hurt existing adopters (who will have generated from the old template), and will improve matters for new users.

no-reply pushed a commit that referenced this issue Jan 23, 2019
Remove references to LatLonType from Solr schema
f6e404f 
This type is deprecated in Solr. Its removal from the template here shouldn't
affect existing adopters who may be using it. It will be removed only for folks
running the generator after this commit.

Fixes #1370
@no-reply no-reply mentioned this issue Jan 23, 2019
Merged
bess pushed a commit that referenced this issue Apr 3, 2019
@bess
Remove references to LatLonType from Solr schema
27a9ef5 
This type is deprecated in Solr. Its removal from the template here shouldn't
affect existing adopters who may be using it. It will be removed only for folks
running the generator after this commit.

Fixes #1370
@bess bess closed this as completed in #1376 Apr 3, 2019
bess pushed a commit that referenced this issue Apr 3, 2019
@bess
Remove references to LatLonType from Solr schema
03780e0 
This type is deprecated in Solr. Its removal from the template here shouldn't
affect existing adopters who may be using it. It will be removed only for folks
running the generator after this commit.

Fixes #1370
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jcoyne @no-reply