Berin Loritsch wrote:

> Leif Mortenson wrote:
> 
> 
>>I noticed a couple places in the pool package where classes are not
>>implementing Initializable, when they should.  Wanted to make sure that
>>these were not by design before making any changes.
>>
>>1) The SoftResourceLimitingPool relies on its initialize() method being
>>called, but the class does not currently implement Initializable.
>>HardResourceLimitingPool currently extends SoftResourceLimitingPool and
>>does implement Initialize. It should be removed from the later.
>>
> 
> 
> This was to address a limitation that I can't remember what it is right
> now.  The goal was to move toward an Initializable pool.  However there
> are environments where the user just wants to create the pool and be
> done with it.  It is a Subversion of Control to directly manage your
> own lifecycle classes though.
> 
> My take on it is that I am ready to implement new light-weight
> asynchronously managed pools.  I started this in Scratchpad.  There is
> the FixedSizePool for hard limits (it does not resize making the
> implementation simple and quick).  The VariableSizedPool can be
> externally managed.


Ok, that sounds good.


About the SingleThreadedPool though.  It had an additional problem with
its interaction with the ObjectFactory.  Created Objects were never
being decommissioned.  I fixed those problems, but had to add a
Disposable interface and make it extend the AbstractLogEnabled class so
that I had something to do with the exceptions thrown by the
decommission method.
The change in the API should not affect anyone using the class as I
implemented it in such a way that the class will behave correctly even
if the code using the class does not call enableLogging() and disable()
in turn.

Cheers,
Leif


> 
> 
> 
> 
> 
> ----------------------------------------------------
> Sign Up for NetZero Platinum Today
> Only $9.95 per month!
> http://my.netzero.net/s/signup?r=platinum&refcd=PT97
> 
> --
> To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
> 
> 
> 



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

Reply via email to