Re: trivial smtpd scheduler diff

2012-07-17 Thread Gilles Chehade
mh, is it really that verbose ?

Gilles


On Tue, Jul 17, 2012 at 06:54:41PM +0100, percy piper wrote:
 Any objections to making the scheduler a little less verbose?
 
 Cheers,
 Percy.
 
 Index: scheduler.c
 ===
 RCS file: /cvs/src/usr.sbin/smtpd/scheduler.c,v
 retrieving revision 1.6
 diff -u -p -r1.6 scheduler.c
 --- scheduler.c   10 Jul 2012 11:13:40 -  1.6
 +++ scheduler.c   17 Jul 2012 17:48:00 -
 @@ -355,7 +355,7 @@ scheduler_timeout(int fd, short event, v
   return;
 
  scheduler_sleep:
 - log_info(scheduler: sleeping);
 + log_trace(TRACE_SCHEDULER, scheduler: sleeping);
   return;
  }
 

-- 
Gilles Chehade

https://www.poolp.org  @poolpOrg



Re: trivial smtpd scheduler diff

2012-07-17 Thread percy piper
I'd say twice per message processed is unnecessarily verbose but I
assume you want it that way for a reason.

It is a trivial niggle and probably irritates only me :-)

Thanks for looking Gilles.

Percy.

On 17 July 2012 19:04, Gilles Chehade gil...@poolp.org wrote:
 mh, is it really that verbose ?



Re: trivial smtpd scheduler diff

2012-07-17 Thread Gilles Chehade
On Tue, Jul 17, 2012 at 07:32:38PM +0100, percy piper wrote:
 I'd say twice per message processed is unnecessarily verbose but I
 assume you want it that way for a reason.
 

ok that's too verbose indeed.


 It is a trivial niggle and probably irritates only me :-)
 

no i can understand, i usually have lots of verbose logs because I run
in debug mode I wasn't aware that it was THAT verbose otherwise ;-)

-- 
Gilles Chehade

https://www.poolp.org  @poolpOrg