No worries. I think it's pretty impressive how you spot these things.
-David
On Jan 7, 2008, at 10:00 PM, Manu George wrote:
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
