Kees Jongenburger wrote:
> > I'm afraid you might be wrong. Did you also remove the
> > kicker.setPriority(Thread.MIN_PRIORITY+1) from the
> > EmailBackgroundHandler.java between 1.7.0 and 1.7.1 ? This means that
> > the background email no longer runs in a thread with low priority in
> > 1.7.1. I will have to warn the editors that the next email newsleter
> > will make the website unavailable ...
> It looks like the email background handler creates a thread for every
> mail that has to be sent so even a Thread.sleep won't help, are you
> sending a lot of mail this way?
> and why isn't  mail queued?
> 
> > I thought that 1.7.1 is a bugfixing release. Removing
> > Thread.MIN_PRIORITY+1 code between 1.7.0 and 1.7.1 doesn't sound like
> > bugfixing to me.

'a year ago' is not beteen 1.7.0 and 1.7.1

> 
> It depends on the outcome of this discussion i think. But I assume
> there are plenty bugs left in the email module

I actually do think that Threads like those of crontab must run on
MIN_PRIORITY. I don't quite get the point of that +1 though. Is there any
rationale for it?

Michiel


-- 
Michiel Meeuwissen                  mihxil'
Mediacentrum 140 H'sum                [] ()
+31 (0)35 6772979         nl_NL eo_XX en_US




Reply via email to