Re: [Geotools-devel] Upgrading our connection pool library

2014-06-04 Thread Andrea Aime
On Wed, Jun 4, 2014 at 3:19 AM, Ben Caradoc-Davies ben.caradoc-dav...@csiro.au wrote: So DBCP 1.4 does not support minEvictableIdleTimeMillis/timeBetweenEvictionRunsMillis or equivalent? http://docs.geoserver.org/latest/en/user/data/app- schema/data-stores.html#jndi Those are supported,

Re: [Geotools-devel] Upgrading our connection pool library

2014-06-04 Thread Ben Caradoc-Davies
What about minIdle=0? That seemed to fix my problems with firewalls between GeoServer and the database. (Still supporting upgrade to DBCP 2.0.) On 04/06/14 14:35, Andrea Aime wrote: Those are supported, but they are not useful, as the pool won't evict connections below the minIdle threshold,

Re: [Geotools-devel] Upgrading our connection pool library

2014-06-04 Thread Andrea Aime
On Wed, Jun 4, 2014 at 8:37 AM, Ben Caradoc-Davies ben.caradoc-dav...@csiro.au wrote: What about minIdle=0? That seemed to fix my problems with firewalls between GeoServer and the database. Yes, that combination would work, but the cure will imho damage the patient in the process, as it will

Re: [Geotools-devel] Upgrading our connection pool library

2014-06-04 Thread Ben Caradoc-Davies
Connections must be idle for at least 5 minutes (and up to 10) with these settings: minEvictableIdleTimeMillis=30 timeBetweenEvictionRunsMillis=30 If requests are very infrequent, then I agree that they will be much slower. For more frequent requests there should be no loss of

Re: [Geotools-devel] Upgrading our connection pool library

2014-06-04 Thread Andrea Aime
On Wed, Jun 4, 2014 at 8:48 AM, Ben Caradoc-Davies ben.caradoc-dav...@csiro.au wrote: Connections must be idle for at least 5 minutes (and up to 10) with these settings: minEvictableIdleTimeMillis=30 timeBetweenEvictionRunsMillis=30 If requests are very infrequent, then I agree

[Geotools-devel] [jira] (GEOT-4812) Table hints not applied to aggregate queries

2014-06-04 Thread Andrea Aime (JIRA)
Title: Message Title Andrea Aime created an issue

[Geotools-devel] Build failed in Jenkins: GeoTools-Master #281

2014-06-04 Thread winbuild
See http://winbuild.geo-solutions.it/jenkins/job/GeoTools-Master/281/changes Changes: [andrea aime] [GEOT-4812] Table hints not applied to aggregate queries -- [...truncated 17453 lines...] [JENKINS] Archiving

[Geotools-devel] wfs-ng breaking windows build

2014-06-04 Thread Andrea Aime
Hi, the wfs-ng module has been breaking the windows build server for a while now. Shall we take it out of the build? Example failure: testInit_1_0(org.geotools.data.wfs.internal.WFSClientTest) Time elapsed: 0 sec ERROR! java.io.FileNotFoundException:

Re: [Geotools-devel] wfs-ng breaking windows build

2014-06-04 Thread Jody Garnett
Yeah, I put it in after asking Niels. It can be added back after his pull request. Jody Garnett On Wed, Jun 4, 2014 at 7:53 PM, Andrea Aime andrea.a...@geo-solutions.it wrote: Hi, the wfs-ng module has been breaking the windows build server for a while now. Shall we take it out of the

[Geotools-devel] [jira] (GEOT-4813) Add pagination support to UniqueVisitor

2014-06-04 Thread Mauro Bartolomeoli (JIRA)
Title: Message Title Mauro Bartolomeoli created an issue

Re: [Geotools-devel] wfs-ng improvements

2014-06-04 Thread Justin Deoliveira
Potentially but at this point we have to prioritize the tasks. If the other issues brought up in this thread are blockers to upgrade to supported status then I think its a better use of time to focus on those. On Tue, Jun 3, 2014 at 4:09 PM, Jody Garnett jody.garn...@gmail.com wrote: One more

Re: [Geotools-devel] Semantics of features aggregation and pagination

2014-06-04 Thread Mauro Bartolomeoli
Hi, I prepared a pull request for the subject: https://github.com/geotools/geotools/pull/470 I used tha approach described in my last email. As an additional note, I also added a preserveOrder flag to help pagination be consistent. With this flag the visitor set mantains the order in which items