fix wagon http lightweight

2011-08-24 Thread nicolas de loof
I'm testing a fix for WAGON-346 and WAGON-347 Before I commit anything to svn I'd like your advice : see https://github.com/ndeloof/maven-wagon/tree/wagon-1.x The idea to make LightWeightHttpWagon thread-safe d

Re: Improvement in wagon-http (use http connection pooling from ASF httpclient)

2011-08-24 Thread Brett Porter
On 24/08/2011, at 6:26 PM, Olivier Lamy wrote: > Hello Folks, > I have just loaded an issue [1] regarding an improvement for wagon > http (the wagon http which use asf httpclient). > This small improvement will simply add a connection pooling mechanism > to avoid http(s) connection for artifacts

Re: Improvement in wagon-http (use http connection pooling from ASF httpclient)

2011-08-24 Thread Olivier Lamy
Because wagons are per-lookup beans and not singleton. So to enable this feature, the pooled connection manager must a static field or available in a singleton component (to be shared between all wagon http instances) and initialised only once time. That's why the flag cannot be configurable tru a

Re: Improvement in wagon-http (use http connection pooling from ASF httpclient)

2011-08-24 Thread nicolas de loof
+1 to set this as default why use system properties to set this flag and not use a per-instance configuration ? 2011/8/24 Olivier Lamy > Hello Folks, > I have just loaded an issue [1] regarding an improvement for wagon > http (the wagon http which use asf httpclient). > This small improvement wi

Improvement in wagon-http (use http connection pooling from ASF httpclient)

2011-08-24 Thread Olivier Lamy
Hello Folks, I have just loaded an issue [1] regarding an improvement for wagon http (the wagon http which use asf httpclient). This small improvement will simply add a connection pooling mechanism to avoid http(s) connection for artifacts requests. My first idea was to not enable it by default bu