Nick Airey wrote:
> Hello everyone,
> 
> 
> I have also posted this question on cocoon-users today - apologies to
> those who have seen this there:
> 
> 
> Currently AbstractAction doesn't implement Poolable (itself or in base
> classes). That explains why there are no examples of pooled actions
> in the current sample sitemap.  Also, the Poolable interface is simply a
> marker, so it is trivial to make my action "poolable". 
> 
> So I did a quick test by simply:
> 
> a) making my action implement Poolable, and
> b) adding pool-max, pool-min, pool-grow to the sitemap definition for
> that action
> 
> 
> And it works.  :-)
> 
> (well, at least, it seems to work. I can see the pool being created, and
> an element being retrieved as needed) My other actions are
> created/destroyed every time.
> 
> 
> So does anyone have any ideas why say AbstractAction doesn't implement
> poolable? Is there something I'm missing that will bite me?


It used to be THreadSafe.  In fact, most of the actions should be marked
ThreadSafe--that way there is only one instance.  The AbstractAction
should not implement any lifestyle interfaces at all.  Each concrete
action should decide its lifestyle directly.


-- 

"They that give up essential liberty to obtain a little temporary safety
  deserve neither liberty nor safety."
                 - Benjamin Franklin


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, email: [EMAIL PROTECTED]

Reply via email to