Mailing-List: contact dev-help@activemq.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@activemq.apache.org
Message-ID: <1412350280.1205318795042.JavaMail.jira@brutus>
Date: Wed, 12 Mar 2008 03:46:35 -0700 (PDT)
From: "Manu T George (JIRA)" <jira@apache.org>
To: dev@activemq.apache.org
Subject: [jira] Created: (AMQ-1618) Improper handling of container
 restrictions in creation of sessions
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

Improper handling of container restrictions in creation of sessions
-------------------------------------------------------------------

                 Key: AMQ-1618
                 URL: https://issues.apache.org/activemq/browse/AMQ-1618
             Project: ActiveMQ
          Issue Type: Bug
          Components: Connector, Geronimo Integration
    Affects Versions: 5.0.0, 4.1.1
         Environment: All
            Reporter: Manu T George
             Fix For: 4.1.2, 5.1.0, 5.2.0


In the getServerSession() method of org.apache.activemq.ra.ServerSessionPoolImpl there is an issue with the code block given below

 ServerSessionImpl ss = createServerSessionImpl();
            // We may not be able to create a session due to the container
            // restricting us.
            if (ss == null) {
                if (idleSessions.size() == 0) {
                    throw new JMSException("Endpoint factory did not allows to any endpoints.");
                }

                return getExistingServerSession();
            }

If the container restricts the creation of a session then messages should be queued to an existing session. So the check should be 

                if (activeSessions.size() == 0) {
                    throw new JMSException("Endpoint factory did not allows to any endpoints.");
                }

instead of the one above


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.