ngageoint / elasticgeo

ElasticGeo provides a GeoTools data store that allows geospatial features from an Elasticsearch index to be published via OGC services using GeoServer.
GNU General Public License v3.0
168 stars 85 forks source link

Adding support for dates stored as longs in ELS #6

Closed peterneubauer closed 9 years ago

sjudeng commented 9 years ago

There are some test failures in this branch (see below). Can you take a look?

https://travis-ci.org/ngageoint/elasticgeo/jobs/66540386

peterneubauer commented 9 years ago

Should be working now.

sjudeng commented 9 years ago

Thanks for fixing the tests (and adding some!). I'd like to look more to make sure queries on inner object type mappings would still work as expected with this change.

https://github.com/peterneubauer/elasticgeo-1/blob/master/gt-elasticsearch/src/main/java/mil/nga/giat/data/elasticsearch/FilterToElastic.java#L614-L616

I'd expect that a PropertyIsEqual filter, for example, would return a term filter or nested filter depending on whether the relevant inner field has object type mapping or nested type mapping, respectively.