Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-23 Thread Andrea Aime
Hi Ivan, I believe you're going with a custom path where a general one could be used, and the changes can be limited to GeoServer. In the case of vector data I believe only min and max values are extracted even in the case of min/max/resolution, the code path starts here: https://github.com/geoser

Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-23 Thread Ivan Martynovskyi
Hello. As mentioned before, there are two problems with the WMS GetCapabilities operation. The first problem is that one of the wrappers (OracleDatastoreWrapper or PostgisDatastoreWrapper) between GeoServer and the datastore is not passing down the visitor (MinVisitor, MaxVisitor, UniqueVisito

Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-10 Thread Jody Garnett
We should move this to geoserver-devel ... I kind of hope we can configure xstream to smooth of the edges here. -- Jody Garnett On Fri, 10 Aug 2018 at 09:53, Andrea Aime wrote: > Ticked opened: > https://osgeo-org.atlassian.net/browse/GEOS-8884 > > Mind, even if the data dir reading gets addres

Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-10 Thread Andrea Aime
Ticked opened: https://osgeo-org.atlassian.net/browse/GEOS-8884 Mind, even if the data dir reading gets addressed, the change will still break REST clients. The above attribute structure is only present sometimes in the serialized data dir, but it's always present in the REST output for feature ty

Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-10 Thread Andrea Aime
On Fri, Aug 10, 2018 at 6:37 PM Jody Garnett wrote: > Good find Andrea: > > Can you report a GeoServer bug for the featuretype.xml issue and we can > sort out an approach. As with the GEOT-6087 > I would prefer to > store the geometry type, rathe

Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-10 Thread Jody Garnett
Good find Andrea: Can you report a GeoServer bug for the featuretype.xml issue and we can sort out an approach. As with the GEOT-6087 I would prefer to store the geometry type, rather than the class name if we are making a change. Aside: This wou

Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-10 Thread Andrea Aime
Speaking of serializing class names, the GeoServer configuration migth also do that: https://github.com/geoserver/geoserver/blob/6e9e25c0c7cdda9ada9f33f8255130d3afc76801/src/main/src/main/java/org/geoserver/catalog/AttributeTypeInfo.java#L71 (see getBinding). However, I don't remember under what c

Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-10 Thread Jody Garnett
Thanks for the report Gyorgy, this was overlooked in the upgrade process - we were quite focused on serialization, but did not think about any of our formats recording class names (usually you would record the geometry type which is a shorter string). Started an issue here https://osgeo-org.atlass

Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-10 Thread György Tomcsányi
On 10. 8. 2018 15:00, Andrea Aime wrote: On Fri, Aug 10, 2018 at 2:50 PM György Tomcsányi > wrote: Hi Andrea, we are preparing the pull request. We had a small problem when upgrading Geoserver to the latest build to test it. We had e

Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-10 Thread Andrea Aime
On Fri, Aug 10, 2018 at 2:50 PM György Tomcsányi < gyorgy.tomcsa...@microstep-mis.com> wrote: > Hi Andrea, > > we are preparing the pull request. > > We had a small problem when upgrading Geoserver to the latest build to > test it. We had existing mosaics in the data dir, and Geoserver failed to >

Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-10 Thread György Tomcsányi
Hi Andrea, we are preparing the pull request. We had a small problem when upgrading Geoserver to the latest build to test it. We had existing mosaics in the data dir, and Geoserver failed to start because of the JTS version upgrade. File /.mosaic/.properties contains a reference to "com.vivi

Re: [Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-09 Thread Andrea Aime
Ah, by looking at it, it would seems the store wrapper in the mosaic module is not correctly delegating down the visitor and thus breaking database optimizations. Yep, pull requests are quite welcomed, please pay attention to the rules to contribute, detailed here: https://github.com/geotools/geoto

[Geotools-devel] ImageMosaic GetCapabilities performance

2018-08-09 Thread György Tomcsányi
Hello all, we are using GeoServer to display a large number of GeoTIFFs using ImageMosaic data stores. The data has several dimensions (time and custom). We are adding new data to these stores periodically and the goal is to be able to display years of data (millions of granules). We have enc