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 < [email protected]> 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] dependency to Jdk14Logger impl

2018-08-10 Thread Frank Gasdorf via GeoTools-Devel
Ok, I've created one. Moving discussion to issue : https://osgeo-org.atlassian.net/browse/GEOT-6084 Am Do., 9. Aug. 2018 um 15:11 Uhr schrieb Jody Garnett < [email protected]>: > Raise a bug, and we need to review these dependencies for Java 18.9 > compatibility. > On Wed, Aug 8, 2018 at 10: