Re: [PATCH 1/2] MINOR: add be_conn_free sample fetch

2018-08-24 Thread Patrick Hemmer
On 2018/8/25 01:30, Willy Tarreau wrote: > On Fri, Aug 24, 2018 at 06:18:23PM -0400, Patrick Hemmer wrote: >>> I disagree with making a special case above for maxconn 0. In fact for me >>> it just means that such a server cannot accept connections, so it simply >>> doesn't count in the sum, just

Re: [PATCH 1/2] MINOR: add be_conn_free sample fetch

2018-08-24 Thread Willy Tarreau
On Fri, Aug 24, 2018 at 06:18:23PM -0400, Patrick Hemmer wrote: > > I disagree with making a special case above for maxconn 0. In fact for me > > it just means that such a server cannot accept connections, so it simply > > doesn't count in the sum, just as if it were not present. > On a server, max

Re: [PATCH 1/2] MINOR: add be_conn_free sample fetch

2018-08-24 Thread Patrick Hemmer
On 2018/8/22 04:04, Willy Tarreau wrote: > Hi Patrick, > > On Thu, Aug 09, 2018 at 06:46:28PM -0400, Patrick Hemmer wrote: >> This adds the sample fetch 'be_conn_free([])'. This sample fetch >> provides the total number of unused connections across available servers >> in the specified backend. >

Re: [PATCH 1/2] MINOR: add be_conn_free sample fetch

2018-08-22 Thread Willy Tarreau
Hi Patrick, On Thu, Aug 09, 2018 at 06:46:28PM -0400, Patrick Hemmer wrote: > > This adds the sample fetch 'be_conn_free([])'. This sample fetch > provides the total number of unused connections across available servers > in the specified backend. Thanks for writing this one, I recently figured