Re: [PATCH] Reduce logging in dbpool_mysql.c

2016-10-26 Thread amalysh
t; Alexander Malysh > > From: Stipe Tolj <st...@kannel.org> <mailto:st...@kannel.org> > Sent: Monday, September 26, 2016 07:35 PM > To: > Cc: kannel_dev_mailinglist <devel@kannel.org> <mailto:devel@kannel.org> > Subject: Re: [PATCH] Reduce loggin

Re: [PATCH] Reduce logging in dbpool_mysql.c

2016-10-18 Thread Donald Jackson
er 26, 2016 07:35 PM > *To*: > *Cc*: kannel_dev_mailinglist <devel@kannel.org> <devel@kannel.org> > *Subject*: Re: [PATCH] Reduce logging in dbpool_mysql.c > Am 24.09.2016 13:57, schrieb Rene Kluwen: > > I don’t agree completely with your reasoning. > > > > Ei

RE: [PATCH] Reduce logging in dbpool_mysql.c

2016-10-18 Thread Alexander Malysh
ysh   From: Stipe Tolj <st...@kannel.org> Sent: Monday, September 26, 2016 07:35 PM To: Cc: kannel_dev_mailinglist <devel@kannel.org> Subject: Re: [PATCH] Reduce logging in dbpool_mysql.c Am 24.09.2016 13:57, schrieb Rene Kluwen: > I don’t agree completely with your reasoning. > &g

Re: [PATCH] Reduce logging in dbpool_mysql.c

2016-09-26 Thread Stipe Tolj
Am 24.09.2016 13:57, schrieb Rene Kluwen: I don’t agree completely with your reasoning. Either you want debug information or you don’t. Enabling debug information yields a lot of information. What about if you want to debug the mysql pool usage? Still +0 from me in case it’s redundant info.

Re: [PATCH] Reduce logging in dbpool_mysql.c

2016-09-24 Thread Donald Jackson
I'm on your side but many Kannel users run in debug log level, and this particular information is the same every time - it doesn't change from request to request. It's a non essential patch just 'nice to have' for most. On Saturday, 24 September 2016, Rene Kluwen wrote: >

RE: [PATCH] Reduce logging in dbpool_mysql.c

2016-09-24 Thread Rene Kluwen
I don’t agree completely with your reasoning. Either you want debug information or you don’t. Enabling debug information yields a lot of information. What about if you want to debug the mysql pool usage? Still +0 from me in case it’s redundant info. Van: devel