Lat/Long are Reversed In GeoDistanceFilterBuilder

[darylrobbins]

The latitude and longitude values are being reversed by the GeoDistanceFilterBuilder when attempted to construct a search. I have had to do the following in my client code:

searchClient.prepareSearch("stores")
        .setTypes("store")
        .setQuery(QueryBuilders.matchAllQuery())
        .setFilter(FilterBuilders.geoDistanceFilter("location")
        .lat(longitude)          // BUG: These are reversed in the ES API
        .lon(latitude)

The issue is with the following line of code:
https://github.com/elasticsearch/elasticsearch/blob/master/src/main/java/org/elasticsearch/index/query/GeoDistanceFilterBuilder.java#L125

Fixing this issue would however break the backwards compatibility of the API, if anyone happened to be relying on this functionality being reversed to get their application working.

[kimchy]

Its not reversed, right? When using the array format, we conform to the geojson spec, and do [lon, lat].

[darylrobbins]

Well, that explains it. Thanks!