Hi You were right Jukka, when changing mechanism to retrieve all geometric layers information at once, I missed a step to set information whether a geometric column is indexed or not. GeometryColumn where built with a default value of False for isIndex attribute.
I fixed the code and now spatial index is used, either for spatialite or geospatial package. Furthermore, another bug prevented the default spatial filter to be applied for spatialite datasources, leading to a full layer load even for a small view bounding box. (tested On a 800K polygons dataset: several seconds to load a small area with the bug, 25ms after correction and use of spatial index) Nicolas On Wed, 12 Feb 2020 at 13:20, <edgar.sol...@web.de> wrote: > sounds good!.. ede > > On 12.02.2020 12:27, Nicolas Ribot wrote: > > Hi, > > > > I'm currently looking at the problem, that also occur with spatialite > > format. > > I hope it will be fixed in few hours. > > I'll keep you posted. > > > > Nicolas > > > > On Wed, 12 Feb 2020 at 11:53, <edgar.sol...@web.de> wrote: > > > >> hey Nico, > >> > >> would you like me to hold the release for the fix or do you need more > >> time? .. snappy new one btw. ede > >> > >> > >> On 12.02.2020 10:57, Nicolas Ribot wrote: > >>> Hi, > >>> > >>> I will have a look at it ! > >>> > >>> Nicolas > >>> > >>> On Wed, 12 Feb 2020 at 08:31, Rahkonen Jukka (MML) < > >>> jukka.rahko...@maanmittauslaitos.fi> wrote: > >>> > >>>> Hi, > >>>> > >>>> > >>>> > >>>> It seems that loosing spatial index of GeoPackage tables is a side > >> effect > >>>> of this changeset that was otherwise a nice improvement > >>>> https://sourceforge.net/p/jump-pilot/code/5976/ > >>>> > >>>> > >>>> > >>>> -Jukka- > >>>> > >>>> > >>>> > >>>> *Lähettäjä:* Rahkonen Jukka (MML) < > jukka.rahko...@maanmittauslaitos.fi> > >>>> *Lähetetty:* keskiviikko 12. helmikuuta 2020 0.14 > >>>> *Vastaanottaja:* jump-pilot-devel@lists.sourceforge.net > >>>> *Aihe:* [JPP-Devel] Deprecation with GeoPackage and spatial index > >>>> > >>>> > >>>> > >>>> Hi, > >>>> > >>>> > >>>> > >>>> I noticed that there has happened some deprecation with GeoPackages > >> since > >>>> released OpenJUMP 1.12. Version 1.12 detects and utilizes the spatial > >> index > >>>> of a table which makes it very fast to use even huge tables if map is > >>>> zoomed close enough. New snapshots do not utilize spatial index which > >>>> makes OJ very slow with large tables. I have not yet analyzed more > >> closely > >>>> when this change has happened. It seems that the layer list is sorted > >>>> differently now than in 1.12. Perhaps that change is related. > >>>> > >>>> > >>>> > >>>> I believe that the detection is done here > >>>> > >> > https://sourceforge.net/p/jump-pilot/code/HEAD/tree/core/trunk/src/com/vividsolutions/jump/datastore/spatialite/SpatialiteDSMetadata.java#l151 > >>>> . > >>>> > >>>> > >>>> > >>>> -Jukka Rahkonen- > >>>> > >>> > >>> > >>> > >>> _______________________________________________ > >>> Jump-pilot-devel mailing list > >>> Jump-pilot-devel@lists.sourceforge.net > >>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > >>> > >> > >> > > > > > > _______________________________________________ > Jump-pilot-devel mailing list > Jump-pilot-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >
_______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
