Thanks for clarifying and adding the comments David.I guess the current approach makes the most sense. So I will close the JIRA in question.
Thanks Manu On Jan 8, 2008 3:48 AM, David Blevins <[EMAIL PROTECTED]> wrote: > > On Jan 7, 2008, at 2:44 AM, Manu George wrote: > > > Hi, > > I opened a JIRA for this but I guess my initial understanding > > may not be right. > > > > Section 14.3.1 of the spec has contradictions to what I stated :(. > > > > [C] Discard instance means that the container must not invoke any > > business methods or container callbacks on the instance. > > > > Now I am not sure what is the correct way. Also Note[C] is mentioned > > only in some cases of discard instance. Do the other discard instances > > not mean this? Very confused now. > > Great discussion. > > Well we've applied it across the board to all bean types. > > In the case of Stateless beans, an instance is removed from the pool > (via StatelessInstanceManager.getInstance) before we invoke the > business method to ensure only one thread is executing that instance. > If the business method throws a system exception, we simply don't put > it back into the pool (would be via > StatelessInstanceManager.poolInstance) and it should eventually be > garbage collected. > > I added some comments to that affect in the StatelessContainer and > StatelessInstanceManager. > > -David > >
