Skip to content

GeoJson types can be lowercase in Elasticsearch. #1655

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.

Sign up for GitHub

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
Closed
sothawo opened this issue Jan 19, 2021 · 0 comments · Fixed by #1657
Closed

GeoJson types can be lowercase in Elasticsearch. #1655

sothawo opened this issue Jan 19, 2021 · 0 comments · Fixed by #1657
Labels
in: mapping Mapping and conversion infrastructure type: bug A general bug
Milestone
4.2 M3 (2021.0.0)

Comments

@sothawo
Copy link
Collaborator

sothawo commented Jan 19, 2021 •
edited
Loading

The GeoJson classes have a type property, for example Point or MultiLineString. In the classes these are camel-case names as defined in the GeoJson specification.

Elasticsearch can use both camelcase and lowercase names for the types. This leads to an error when documents are returned from Elasticsearch that contains type names in lowercase.

Spring Data Elasticsearch should be able to read the documents regardless of the camelcase or lowercase naming.
@sothawo sothawo added type: bug A general bug in: mapping Mapping and conversion infrastructure labels Jan 19, 2021
@sothawo sothawo changed the title GeoJson types must be lowercase in Elasticsearch. GeoJson types can be lowercase in Elasticsearch. Jan 19, 2021
@sothawo sothawo mentioned this issue Jan 19, 2021
Merged
@sothawo sothawo added this to the 4.2 M3 (2021.0.0) milestone Jan 19, 2021
sothawo added a commit that referenced this issue Jan 19, 2021
@sothawo
GeoJson types can be lowercase in Elasticsearch.
159520d 
Original Pull Request #1657 
Closes #1655
sothawo added a commit that referenced this issue Jan 19, 2021
@sothawo
GeoJson types can be lowercase in Elasticsearch.
d35d38f 
Original Pull Request #1657
Closes #1655

(cherry picked from commit 159520d)
@sothawo sothawo mentioned this issue Jan 19, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
in: mapping Mapping and conversion infrastructure type: bug A general bug
Projects
None yet
Milestone
4.2 M3 (2021.0.0)
Development

Successfully merging a pull request may close this issue.

GeoJson types can be lowercase in Elasticsearch. sothawo/spring-data-elasticsearch
1 participant
@sothawo