[ http://jira.jboss.com/jira/browse/JBAS-1556?page=history ]
     
Scott M Stark closed JBAS-1556:
-------------------------------

      Assign To: Scott M Stark
     Resolution: Done
    Fix Version: JBossAS-4.0.2 Final
                 JBossAS-5.0 Alpha

The changes have been integrated.

> SingletonStatelessSessionInstancePool doesn't work properly now
> ---------------------------------------------------------------
>
>          Key: JBAS-1556
>          URL: http://jira.jboss.com/jira/browse/JBAS-1556
>      Project: JBoss Application Server
>         Type: Patch
>   Components: EJBs
>     Versions: JBossAS-4.0.1 Final
>     Reporter: Oleg Nitz
>     Assignee: Scott M Stark
>     Priority: Minor
>      Fix For: JBossAS-5.0 Alpha, JBossAS-4.0.2 Final

>
>
> Index: SingletonStatelessSessionInstancePool.java
> ===================================================================
> RCS file: 
> /cvsroot/jboss/jboss/src/main/org/jboss/ejb/plugins/SingletonStatelessSessionInstancePool.java,v
> retrieving revision 1.23
> diff -c -r1.23 SingletonStatelessSessionInstancePool.java
> *** SingletonStatelessSessionInstancePool.java        27 Aug 2003 04:32:04 
> -0000      1.23
> --- SingletonStatelessSessionInstancePool.java        9 Mar 2005 16:53:42 
> -0000
> ***************
> *** 30,43 ****
>    *  @author <a href="mailto:[EMAIL PROTECTED]">Rickard ?berg</a>
>    *  @version $Revision: 1.23 $
>    */
> ! public class SingletonStatelessSessionInstancePool
> !    implements InstancePool, XmlLoadable
>   {
>      // Constants -----------------------------------------------------
>   
>      // Attributes ----------------------------------------------------
> -    Container con;
> - 
>      EnterpriseContext ctx;
>      boolean inUse = false;
>      boolean isSynchronized = true;
> --- 30,40 ----
>    *  @author <a href="mailto:[EMAIL PROTECTED]">Rickard ?berg</a>
>    *  @version $Revision: 1.23 $
>    */
> ! public class SingletonStatelessSessionInstancePool extends 
> AbstractInstancePool
>   {
>      // Constants -----------------------------------------------------
>   
>      // Attributes ----------------------------------------------------
>      EnterpriseContext ctx;
>      boolean inUse = false;
>      boolean isSynchronized = true;
> ***************
> *** 48,64 ****
>   
>      // Public --------------------------------------------------------
>   
> -    /**
> -     *   Set the callback to the container. This is for initialization.
> -     *   The pool may extract the configuration from the container.
> -     *
> -     * @param   c
> -     */
> -    public void setContainer(Container c)
> -    {
> -       this.con = c;
> -    }
> - 
>      public void create()
>         throws Exception
>      {
> --- 45,50 ----
> ***************
> *** 97,103 ****
>         {
>            try
>            {
> !             ctx = create(con.createBeanClassInstance(), con);
>            } catch (InstantiationException e)
>            {
>               throw new EJBException("Could not instantiate bean", e);
> --- 83,89 ----
>         {
>            try
>            {
> !             ctx = create(getContainer().createBeanClassInstance());
>            } catch (InstantiationException e)
>            {
>               throw new EJBException("Could not instantiate bean", e);
> ***************
> *** 131,137 ****
>         this.notifyAll();
>      }
>   
> !    public void discard(EnterpriseContext ctx)
>      {
>         // Throw away
>         try
> --- 117,123 ----
>         this.notifyAll();
>      }
>   
> !    public synchronized void discard(EnterpriseContext ctx)
>      {
>         // Throw away
>         try
> ***************
> *** 178,189 ****
>      // Package protected ---------------------------------------------
>   
>      // Protected -----------------------------------------------------
> !    protected EnterpriseContext create(Object instance, Container con)
>         throws Exception
>      {
> !       return new StatelessSessionEnterpriseContext(instance, con);
>      }
> - 
>      // Private -------------------------------------------------------
>   
>      // Inner classes -------------------------------------------------
> --- 164,175 ----
>      // Package protected ---------------------------------------------
>   
>      // Protected -----------------------------------------------------
> !    protected EnterpriseContext create(Object instance)
>         throws Exception
>      {
> !       // The instance is created by the caller and is a newInstance();
> !       return new StatelessSessionEnterpriseContext(instance, 
> getContainer());
>      }
>      // Private -------------------------------------------------------
>   
>      // Inner classes -------------------------------------------------

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.jboss.com/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
JBoss-Development mailing list
JBoss-Development@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to