Steve,

>   2. DefaultConnectionManager - updated sources so that it doesn't throw
> an
>      exception when disconnecting an unknown connection (James is
> disconnecting
>      twice and the second time is resulting in an
> IllegalArgumentException.  I've
>      made a minor update that basically returns if the supplied
> connection is
>      unknown).

More out of curiosity than anything else, why was this change necessary? 

Was this a Merlin thing?  In the standard Phoenix-based configuration, James
doesn't even use DefaultConnectionManager.  There is a specific
ConnectionManager implementation -
org.apache.james.util.connection.SimpleConnectionManager - that is used
instead.  In theory one could use DefaultConnectionManager, but I don't see
why one would, as it lacks some of the connection throttling features of the
James class.

--Peter 



--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to