Use Types.LatLon for location

bitemyapp / bloodhound

Haskell Elasticsearch client and query DSL

bitemyapp.com

BSD 3-Clause "New" or "Revised" License

424 stars 118 forks source link

Use Types.LatLon for location #204

Closed jagare closed 7 years ago

jagare commented 7 years ago

Without this change, code compile but fail against ES, since names for lat lon (in Location record) don't follow specification for geo_point. I changed to use Types.LatLon instead.

There is still a problem with the example regarding the IndexTemplate. However that is not addressed in this PR.

jagare commented 7 years ago

In the tests the same Location record is defined but using the names lat and lon instead here. So the test passes.

However wouldn't it be a good idea to use the LatLon type in the tests instead?

jagare commented 7 years ago

Now I found [PR #198] with an identical solution as far as I can see. Guess this one can be abandoned.