Re: [Geotools-devel] Enabling usage of MultiThreadedHttpClient in WFSDataStoreFactory

2018-03-08 Thread Mauro Bartolomeoli
Hi Andrea, I merged the geotools PR and prepared one for GeoServer ( https://github.com/geoserver/geoserver/pull/2790) to add documentation and UI labels for the new parameter. I also added a bunch that were missing from previous work. Mauro 2018-03-05 11:23 GMT+01:00 Andrea Aime : > On Mon, Ma

Re: [Geotools-devel] Enabling usage of MultiThreadedHttpClient in WFSDataStoreFactory

2018-03-07 Thread Mauro Bartolomeoli
Hi, I prepared a PR for this one: https://github.com/geotools/geotools/pull/1820 If this is ok, I also have one for GeoServer, to update the documentation and UI labels. 2018-03-05 11:35 GMT+01:00 Mauro Bartolomeoli < [email protected]>: > Hi Andrea, all, > I found at least on

Re: [Geotools-devel] Enabling usage of MultiThreadedHttpClient in WFSDataStoreFactory

2018-03-05 Thread Mauro Bartolomeoli
Hi Andrea, all, I found at least one difference: MultiThreaded does not work on the file: protocol, so tests are failing. That said I think it's better to go for option 4. I would also introduce a check to switch to SimpleHttpClient for protocols unsupported by apache-commons (basically anything

Re: [Geotools-devel] Enabling usage of MultiThreadedHttpClient in WFSDataStoreFactory

2018-03-05 Thread Andrea Aime
On Mon, Mar 5, 2018 at 11:05 AM, Mauro Bartolomeoli < [email protected]> wrote: > Hi, > does anyone know why the usage of MultiThreadedHttpClient is comented out > here: > > https://github.com/geotools/geotools/blob/master/modules/ > unsupported/wfs-ng/src/main/java/org/geotools/