[JPP-Devel] SVN: [6648] core/trunk/src

2020-12-11 Thread jump-pilot-svn--- via Jump-pilot-devel 
Revision: 6648
  http://sourceforge.net/p/jump-pilot/code/6648
Author:   elnico
Date: 2020-12-11 16:14:31 + (Fri, 11 Dec 2020)
Log Message:
---
Replaced references to deprecated ST_Estimated_Extent by ST_EstimatedExtent

Modified Paths:
--

core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java

core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java

core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java

Modified: 
core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
===
--- 
core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java 
2020-12-02 16:46:55 UTC (rev 6647)
+++ 
core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java 
2020-12-11 16:14:31 UTC (rev 6648)
@@ -17,7 +17,7 @@
 datasetNameQuery = "SELECT DISTINCT f_table_schema, f_table_name FROM 
geometry_columns";
 defaultSchemaName = "public";
 spatialDbName = "PostGIS";
-spatialExtentQuery1 = "SELECT ST_AsBinary(ST_Estimated_Extent( '%s', 
'%s', '%s' ))";
+spatialExtentQuery1 = "SELECT ST_AsBinary(ST_EstimatedExtent( '%s', 
'%s', '%s' ))";
 // Nicolas Ribot: add double quotes for identifiers
 spatialExtentQuery2 = "SELECT 
ST_AsBinary(ST_Envelope(ST_Extent(\"%s\"))) FROM \"%s\".\"%s\"";
 geoColumnsQuery = "SELECT f_geometry_column, coord_dimension, srid, 
type FROM geometry_columns where f_table_schema='%s' and f_table_name = '%s'";

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
===
--- 
core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
  2020-12-02 16:46:55 UTC (rev 6647)
+++ 
core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
  2020-12-11 16:14:31 UTC (rev 6648)
@@ -55,7 +55,7 @@
 currentQueryContext = myQueryContext;
 
 // [mmichaud 2013-10-20] remove the test using dataset envelope.
-// Indeed, dataset envelope is generally computed using 
ST_Estimated_Extent,
+// Indeed, dataset envelope is generally computed using ST_EstimatedExtent,
 // which is fast, but may be false.
 // Using a false envelope is not too severe as long as it is used in 
functions
 // like "zoom on the whole layer", but if used in this function, it can 
prevent

Modified: 
core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
===
--- 
core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
   2020-12-02 16:46:55 UTC (rev 6647)
+++ 
core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
   2020-12-11 16:14:31 UTC (rev 6648)
@@ -267,7 +267,7 @@
 if (jdbcConn != null) jdbcConn.setAutoCommit(true);
 }
 // Adding vacuum analyze seems to be necessary to be able to 
use
-// ST_Estimated_Extent on the newly created table
+// ST_EstimatedExtent on the newly created table
 //finalizeUpdate(conn);
 }
 
@@ -283,7 +283,7 @@
 /**
  * With some databases, it may be useful to do some cleaning after a big 
update.
  * Example : perform a vacuum analyze in PostgreSQL to compact database 
and to
- * update statistics (needed by ST_Estimated_Extent function)
+ * update statistics (needed by ST_EstimatedExtent function)
  */
 @Deprecated // maybe much time consuming, to be driven by the server, not 
the client
 public abstract void finalizeUpdate(SpatialDatabasesDSConnection conn) 
throws Exception;



___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Re: [JPP-Devel] SVN: [6648] core/trunk/src

2020-12-11 Thread edgar . soldin 
elnico :))

how are you? we are nearing a final OJ1 release to continue with OJ2 based on 
on the latest JTS. we'll also move development into a git repo, at least that's 
the plan at this stage.

any comment or open issues you want to tackle before the release? ..regards ede

On 11.12.2020 17:14, jump-pilot-svn--- via Jump-pilot-devel wrote:
> Revision: 6648
>   http://sourceforge.net/p/jump-pilot/code/6648
> Author:   elnico
> Date: 2020-12-11 16:14:31 + (Fri, 11 Dec 2020)
> Log Message:
> ---
> Replaced references to deprecated ST_Estimated_Extent by ST_EstimatedExtent
>
> Modified Paths:
> --
> 
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
> 
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
> 
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
>
> Modified: 
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
> =
> --- 
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
>2020-12-02 16:46:55 UTC (rev 6647)
> +++ 
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
>2020-12-11 16:14:31 UTC (rev 6648)
> @@ -17,7 +17,7 @@
>  datasetNameQuery = "SELECT DISTINCT f_table_schema, f_table_name 
> FROM geometry_columns";
>  defaultSchemaName = "public";
>  spatialDbName = "PostGIS";
> -spatialExtentQuery1 = "SELECT ST_AsBinary(ST_Estimated_Extent( '%s', 
> '%s', '%s' ))";
> +spatialExtentQuery1 = "SELECT ST_AsBinary(ST_EstimatedExtent( '%s', 
> '%s', '%s' ))";
>  // Nicolas Ribot: add double quotes for identifiers
>  spatialExtentQuery2 = "SELECT 
> ST_AsBinary(ST_Envelope(ST_Extent(\"%s\"))) FROM \"%s\".\"%s\"";
>  geoColumnsQuery = "SELECT f_geometry_column, coord_dimension, srid, 
> type FROM geometry_columns where f_table_schema='%s' and f_table_name = '%s'";
>
> Modified: 
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
> ===
> --- 
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
> 2020-12-02 16:46:55 UTC (rev 6647)
> +++ 
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
> 2020-12-11 16:14:31 UTC (rev 6648)
> @@ -55,7 +55,7 @@
>  currentQueryContext = myQueryContext;
>
>  // [mmichaud 2013-10-20] remove the test using dataset envelope.
> -// Indeed, dataset envelope is generally computed using 
> ST_Estimated_Extent,
> +// Indeed, dataset envelope is generally computed using 
> ST_EstimatedExtent,
>  // which is fast, but may be false.
>  // Using a false envelope is not too severe as long as it is used in 
> functions
>  // like "zoom on the whole layer", but if used in this function, it can 
> prevent
>
> Modified: 
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
> ===
> --- 
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
>  2020-12-02 16:46:55 UTC (rev 6647)
> +++ 
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
>  2020-12-11 16:14:31 UTC (rev 6648)
> @@ -267,7 +267,7 @@
>  if (jdbcConn != null) jdbcConn.setAutoCommit(true);
>  }
>  // Adding vacuum analyze seems to be necessary to be able to 
> use
> -// ST_Estimated_Extent on the newly created table
> +// ST_EstimatedExtent on the newly created table
>  //finalizeUpdate(conn);
>  }
>
> @@ -283,7 +283,7 @@
>  /**
>   * With some databases, it may be useful to do some cleaning after a big 
> update.
>   * Example : perform a vacuum analyze in PostgreSQL to compact database 
> and to
> - * update statistics (needed by ST_Estimated_Extent function)
> + * update statistics (needed by ST_EstimatedExtent function)
>   */
>  @Deprecated // maybe much time consuming, to be driven by the server, 
> not the client
>  public abstract void finalizeUpdate(SpatialDatabasesDSConnection conn) 
> throws Exception;
>
>
>
> ___
> 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

Re: [JPP-Devel] SVN: [6648] core/trunk/src

2020-12-11 Thread Nicolas Ribot 
Hi ede ! :)

I'm fine thank you, I hope you're well also.
Yes I followed the activity on the list and am quite happy with OJ 2 !!

I will have a fresh look on issues and let you know.

Regards
Nicolas

On Fri, 11 Dec 2020 at 17:35,  wrote:

> elnico :))
>
> how are you? we are nearing a final OJ1 release to continue with OJ2 based
> on on the latest JTS. we'll also move development into a git repo, at least
> that's the plan at this stage.
>
> any comment or open issues you want to tackle before the release?
> ..regards ede
>
> On 11.12.2020 17:14, jump-pilot-svn--- via Jump-pilot-devel wrote:
> > Revision: 6648
> >   http://sourceforge.net/p/jump-pilot/code/6648
> > Author:   elnico
> > Date: 2020-12-11 16:14:31 + (Fri, 11 Dec 2020)
> > Log Message:
> > ---
> > Replaced references to deprecated ST_Estimated_Extent by
> ST_EstimatedExtent
> >
> > Modified Paths:
> > --
> >
>  
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
> >
>  
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
> >
>  
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
> >
> > Modified:
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
> > =
> > ---
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
>  2020-12-02 16:46:55 UTC (rev 6647)
> > +++
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
>  2020-12-11 16:14:31 UTC (rev 6648)
> > @@ -17,7 +17,7 @@
> >  datasetNameQuery = "SELECT DISTINCT f_table_schema,
> f_table_name FROM geometry_columns";
> >  defaultSchemaName = "public";
> >  spatialDbName = "PostGIS";
> > -spatialExtentQuery1 = "SELECT ST_AsBinary(ST_Estimated_Extent(
> '%s', '%s', '%s' ))";
> > +spatialExtentQuery1 = "SELECT ST_AsBinary(ST_EstimatedExtent(
> '%s', '%s', '%s' ))";
> >  // Nicolas Ribot: add double quotes for identifiers
> >  spatialExtentQuery2 = "SELECT
> ST_AsBinary(ST_Envelope(ST_Extent(\"%s\"))) FROM \"%s\".\"%s\"";
> >  geoColumnsQuery = "SELECT f_geometry_column, coord_dimension,
> srid, type FROM geometry_columns where f_table_schema='%s' and f_table_name
> = '%s'";
> >
> > Modified:
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
> > ===
> > ---
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
>   2020-12-02 16:46:55 UTC (rev 6647)
> > +++
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
>   2020-12-11 16:14:31 UTC (rev 6648)
> > @@ -55,7 +55,7 @@
> >  currentQueryContext = myQueryContext;
> >
> >  // [mmichaud 2013-10-20] remove the test using dataset envelope.
> > -// Indeed, dataset envelope is generally computed using
> ST_Estimated_Extent,
> > +// Indeed, dataset envelope is generally computed using
> ST_EstimatedExtent,
> >  // which is fast, but may be false.
> >  // Using a false envelope is not too severe as long as it is used
> in functions
> >  // like "zoom on the whole layer", but if used in this function, it
> can prevent
> >
> > Modified:
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
> > ===
> > ---
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
>2020-12-02 16:46:55 UTC (rev 6647)
> > +++
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
>2020-12-11 16:14:31 UTC (rev 6648)
> > @@ -267,7 +267,7 @@
> >  if (jdbcConn != null) jdbcConn.setAutoCommit(true);
> >  }
> >  // Adding vacuum analyze seems to be necessary to be
> able to use
> > -// ST_Estimated_Extent on the newly created table
> > +// ST_EstimatedExtent on the newly created table
> >  //finalizeUpdate(conn);
> >  }
> >
> > @@ -283,7 +283,7 @@
> >  /**
> >   * With some databases, it may be useful to do some cleaning after
> a big update.
> >   * Example : perform a vacuum analyze in PostgreSQL to compact
> database and to
> > - * update statistics (needed by ST_Estimated_Extent function)
> > + * update statistics (needed by ST_EstimatedExtent function)
> >   */
> >  @Deprecated // maybe much time consuming, to be driven by the
> server, not the client
> >  public abstract void finalizeUpdate(SpatialDatabasesDSConnection
> conn) throws Exception;
> >
> >
> >
> > ___
> > Jump-pilot-devel mailing list
> > Jump-pilot-devel@lists.sourceforge.net
> > https://lists.sou

Re: [JPP-Devel] SVN: [6648] core/trunk/src

2020-12-11 Thread edgar . soldin 
On 11.12.2020 17:47, Nicolas Ribot wrote:
> Hi ede ! :)
>
> I'm fine thank you, I hope you're well also.

no complaints apart from the pandemical issue ;)

> Yes I followed the activity on the list and am quite happy with OJ 2 !!

nice

> I will have a fresh look on issues and let you know.

please do. i guess we will make it a traditional end of/new year release again 
at least it's looking a lot like it so far. ..ede

> Regards
> Nicolas
>
> On Fri, 11 Dec 2020 at 17:35,  > wrote:
>
> elnico :))
>
> how are you? we are nearing a final OJ1 release to continue with OJ2 
> based on on the latest JTS. we'll also move development into a git repo, at 
> least that's the plan at this stage.
>
> any comment or open issues you want to tackle before the release? 
> ..regards ede
>
> On 11.12.2020 17:14, jump-pilot-svn--- via Jump-pilot-devel wrote:
> > Revision: 6648
> >           http://sourceforge.net/p/jump-pilot/code/6648 
> 
> > Author:   elnico
> > Date:     2020-12-11 16:14:31 + (Fri, 11 Dec 2020)
> > Log Message:
> > ---
> > Replaced references to deprecated ST_Estimated_Extent by 
> ST_EstimatedExtent
> >
> > Modified Paths:
> > --
> >     
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
> >     
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
> >     
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
> >
> > Modified: 
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
> > =
> > --- 
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
>    2020-12-02 16:46:55 UTC (rev 6647)
> > +++ 
> core/trunk/src/com/vividsolutions/jump/datastore/postgis/PostgisDSMetadata.java
>    2020-12-11 16:14:31 UTC (rev 6648)
> > @@ -17,7 +17,7 @@
> >          datasetNameQuery = "SELECT DISTINCT f_table_schema, 
> f_table_name FROM geometry_columns";
> >          defaultSchemaName = "public";
> >          spatialDbName = "PostGIS";
> > -        spatialExtentQuery1 = "SELECT ST_AsBinary(ST_Estimated_Extent( 
> '%s', '%s', '%s' ))";
> > +        spatialExtentQuery1 = "SELECT ST_AsBinary(ST_EstimatedExtent( 
> '%s', '%s', '%s' ))";
> >          // Nicolas Ribot: add double quotes for identifiers
> >          spatialExtentQuery2 = "SELECT 
> ST_AsBinary(ST_Envelope(ST_Extent(\"%s\"))) FROM \"%s\".\"%s\"";
> >          geoColumnsQuery = "SELECT f_geometry_column, coord_dimension, 
> srid, type FROM geometry_columns where f_table_schema='%s' and f_table_name = 
> '%s'";
> >
> > Modified: 
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
> > ===
> > --- 
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
>         2020-12-02 16:46:55 UTC (rev 6647)
> > +++ 
> core/trunk/src/com/vividsolutions/jump/workbench/model/cache/DynamicFeatureCollection.java
>         2020-12-11 16:14:31 UTC (rev 6648)
> > @@ -55,7 +55,7 @@
> >      currentQueryContext = myQueryContext;
> >
> >      // [mmichaud 2013-10-20] remove the test using dataset envelope.
> > -    // Indeed, dataset envelope is generally computed using 
> ST_Estimated_Extent,
> > +    // Indeed, dataset envelope is generally computed using 
> ST_EstimatedExtent,
> >      // which is fast, but may be false.
> >      // Using a false envelope is not too severe as long as it is used 
> in functions
> >      // like "zoom on the whole layer", but if used in this function, 
> it can prevent
> >
> > Modified: 
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
> > ===
> > --- 
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
>      2020-12-02 16:46:55 UTC (rev 6647)
> > +++ 
> core/trunk/src/org/openjump/core/ui/plugin/datastore/WritableDataStoreDataSource.java
>      2020-12-11 16:14:31 UTC (rev 6648)
> > @@ -267,7 +267,7 @@
> >                      if (jdbcConn != null) jdbcConn.setAutoCommit(true);
> >                  }
> >                  // Adding vacuum analyze seems to be necessary to be 
> able to use
> > -                // ST_Estimated_Extent on the newly created table
> > +                // ST_EstimatedExtent on the newly created table
> >                  //finalizeUpdate(conn);
> >              }
> >
> > @@ -283,7 +283,7 @@
> >      /**
> >       * With some databases, it may be useful to do some cleaning after 
>