Yes good idea, I will add a test before loading a layer. Nicolas
On 12 December 2015 at 20:22, Michaël Michaud <m.michael.mich...@orange.fr> wrote: > Hi Nicolas, > > Thanks for your answer. > > Just one idea about the problem Jukka has mentionned : > Maybe the query should be tested before the Layer is created. > A way to do that would be to add (or replace) the LIMIT clause with value > 0 to have a lightweight query > giving the opportunity to catch a SQLException. > May not be the cleanest way to solve the problem though (several classes > to patch). > > Michaël > > Le 12/12/2015 16:31, Nicolas Ribot a écrit : > > Hi Michaël, > > Thanks for the insight. > > I'm currently refactoring a bit the SpatialDatabases code. > The error you pointed out showed me some problems with current code, and > as you noted, I have to instantiate a correct, concrete class for each > detected DSConnection. > > I'm finalising my todo list, but right now, FilterQuery and AdhocQuery can > be ran on Postgis, Oracle, MySQL and Spatialite databases the same way. > > I'm currently finishing the support for Spatialite, that can be tricky: > geometry column can contain WKT, WKB or native Spatialite binary format. > Still have to correct the ValueConvert to correctly detect and convert each > type. > > MySQL support is added with support for both WKB storage, internal binary > storage (code from DBQuery). > Layer extent is now correctly retrieved but has to be tested on big > datasets (it merely uses a textual aggregation of all geometries to build > the resulting envelope). > > I think I will be able to commit fixes soon. > > I'm stuck, though, on the problem Jukka mentioned: if you add a wrong > where clause to the layer to load, error is detected in > the com.vividsolutions.jump.workbench.ui.renderer.ImageCachingRenderer( > line 90) where the error message is displayed in Window toolbar. > I can display an ErrorDialog with SQL error, but do not know how to > cleanly remove the layer from the panel. > > Nicolas > > > On 12 December 2015 at 15:29, Michaël Michaud <m.michael.mich...@orange.fr > > wrote: > >> Hi Nicolas, >> >> I had a look but did not find an easy way to fix the problem. >> >> In SpatialDatabasesDSConnection you added a comment that it is no more >> necessary to subclass to PostGIS... >> >> On the other hand, >> - SpatialDatabasesDSConnection instantiates >> - SpatialDatabasesResultSetConverter which in turn instantiates >> - SpatialDatabasesFeatureInputStream which instantiates >> - SpatialDatabasesResultSetConverter which instiates a new >> - SpatialDatabasesValueConverterFactory >> >> The getConverter method of the later just return null which seems the >> root cause >> of my NPE. >> >> Seems like the concrete class should still be a subclass of >> SpatialDatabasesDSConnection. >> What do you think ? >> >> Another (maybe related) question : In the component to connect a new >> database, there >> is a combobox with a single value "PostGIS". Should we add other drivers >> in order to >> instantiate proper connection for each driver. There is commented code >> from you in >> JumpConfiguration class showing that you already have implemented such a >> solution >> in the past) >> >> Michaël >> >> Le 08/12/2015 10:30, Nicolas Ribot a écrit : >> >> Hi Michaël, >> >> I will look at it. >> >> Nicolas >> >> On 7 December 2015 at 23:12, Michaël Michaud < >> <m.michael.mich...@orange.fr>m.michael.mich...@orange.fr> wrote: >> >>> Hi >>> >>> Waw, thanks for this refactoring Nicolas ! >>> >>> Just noticed the following regression >>> >>> in the File > Execute Query plugin >>> If I try to query a postgis table as I did before, I get the following >>> NPE : >>> java.lang.NullPointerException >>> at >>> com.vividsolutions.jump.datastore.spatialdatabases.SpatialDatabasesResultSetConverter.init(SpatialDatabasesResultSetConverter.java:72) >>> >>> The same NPE occurs with the read/write postgis driver >>> >>> Any idea ? >>> >>> Michaël >>> >>> >>> Le 07/12/2015 18:16, Nicolas Ribot a écrit : >>> >>> Hi, >>> >>> Oracle Spatial support has been added to OJ core (read-only). >>> It works the same way as Postgis support: define a DB connection to >>> Oracle and choose spatial layers from the list of found layers to display >>> them in OJ >>> >>> Oracle 9i -> 12c supported (though only tested recently with Oracle 11i >>> and 12c). >>> >>> The SpatialDatabases plugin also contains code to support MariaDB/MySQL >>> and Spatialite. (SQL Server in the pipe...) >>> >>> I was thinking about adding this code to the core, too. >>> What do you think ? >>> >>> Nicolas >>> >>> >>> >>> >>> ------------------------------------------------------------------------------ >>> Go from Idea to Many App Stores Faster with Intel(R) XDK >>> Give your users amazing mobile app experiences with Intel(R) XDK. >>> Use one codebase in this all-in-one HTML5 development environment. >>> Design, debug & build mobile apps & 2D/3D high-impact games for multiple >>> OSs.http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140 >>> >>> >>> >>> _______________________________________________ >>> Jump-pilot-devel mailing >>> listJump-pilot-devel@lists.sourceforge.nethttps://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >>> >>> >>> >>> >>> ------------------------------------------------------------------------------ >>> Go from Idea to Many App Stores Faster with Intel(R) XDK >>> Give your users amazing mobile app experiences with Intel(R) XDK. >>> Use one codebase in this all-in-one HTML5 development environment. >>> Design, debug & build mobile apps & 2D/3D high-impact games for multiple >>> OSs. >>> http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140 >>> _______________________________________________ >>> Jump-pilot-devel mailing list >>> Jump-pilot-devel@lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel >>> >>> >> >> >> ------------------------------------------------------------------------------ >> Go from Idea to Many App Stores Faster with Intel(R) XDK >> Give your users amazing mobile app experiences with Intel(R) XDK. >> Use one codebase in this all-in-one HTML5 development environment. >> Design, debug & build mobile apps & 2D/3D high-impact games for multiple >> OSs.http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140 >> >> >> >> _______________________________________________ >> Jump-pilot-devel mailing >> listJump-pilot-devel@lists.sourceforge.nethttps://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 > listJump-pilot-devel@lists.sourceforge.nethttps://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