locationtech / geowave

GeoWave provides geospatial and temporal indexing on top of Accumulo, HBase, BigTable, Cassandra, Kudu, Redis, RocksDB, and DynamoDB.
Apache License 2.0
502 stars 190 forks source link

SpatialTemporalQuery on non-temporal index #1009

Closed rfecher closed 6 years ago

rfecher commented 7 years ago

In this case, the time attribute is ambiguous, but the same inference should be applied as ingesting vector data with unspecified time attributes into a spatial_temporal index: https://github.com/ngageoint/geowave/blob/master/extensions/adapters/vector/src/main/java/mil/nga/giat/geowave/adapter/vector/utils/TimeDescriptors.java#L226

rfecher commented 7 years ago

unless the time attributes are part of the CommonIndexModel the filtering would have to deserialize the simplefeature and it won't be able to run the same as the default SpatialTemporalQuery

Also, perhaps this logic should be generalized to apply to SpatialQuery running on non-spatial index...

For now in all cases CQLQuery.createOptimalQuery() will result in the best query given the index and should be used on SimpleFeature data if unsure: https://github.com/ngageoint/geowave/blob/master/extensions/adapters/vector/src/main/java/mil/nga/giat/geowave/adapter/vector/query/cql/CQLQuery.java#L63