Peter,
> The fact that LinearProcessor matcher/mailets are added before the
> service method is called is not something in the contract of the class.
Maybe it should be; see below.
> synchronizing the add call [costs] essentially nothing
Agreed. The overhead is measurable but insignificant compared to the rest
of the service method.
> Declaring the variables final ...
Go ahead and do it. As I said earlier today, after I thought through the
process of reloading the configuration, it appears to me that the way to do
this is to create a new instance of JamesSpoolManager with the new
configuration, and to gracefully shutdown the old one. Essentially, the old
one would finish up existing message processing, but wouldn't pull new
messages from the spool.
So I'll go you one further. Why not establish that calling
LinearProcessor.add(Matcher,Mailet) after calling LinearProcessor.service()
results in an exception?
--- Noel
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
