Bug 772679 - QpidRAMessage incorrectly calls getSession to determine if underlying JMS session has been closed
Summary: QpidRAMessage incorrectly calls getSession to determine if underlying JMS ses...
Keywords:
Status: CLOSED DUPLICATE of bug 773348
Alias: None
Product: Red Hat Enterprise MRG
Classification: Red Hat
Component: qpid-java
Version: 2.0
Hardware: Unspecified
OS: Unspecified
high
unspecified
Target Milestone: 2.2
: ---
Assignee: messaging-bugs
QA Contact: MRG Quality Engineering
URL: https://issues.apache.org/jira/browse...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-09 16:11 UTC by Martin Vecera
Modified: 2012-02-09 09:44 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-01-11 16:01:12 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)

Description Martin Vecera 2012-01-09 16:11:23 UTC 
Description of problem:
In QpidRASessionImpl prior to returning a reference we evaluate the ConnectionRequestInfo type looking for an XA connection. I believe this code was inadvertently introduced via cut-n-paste and doesn't make sense. Currently it is causing an issue for certain scenarios where we need to manually acknowledge a message via the standard JMS API. This code should be removed.

Additional info:
There is already a patch created by Weston. See https://issues.apache.org/jira/browse/QPID-3731

Impact:
It is not possible to manually call acknowledge() on a message - this leads to an exception - even if the call is valid according to spec.
Comment 1 Weston M. Price 2012-01-11 16:01:12 UTC 
Sorry Martin, I forgot I had already filed this. My bad, you shouldn't have had to file this at all.

*** This bug has been marked as a duplicate of bug 733348 ***
Comment 2 Martin Vecera 2012-02-09 09:44:17 UTC 

*** This bug has been marked as a duplicate of bug 773348 ***
Note You need to log in before you can comment on or make changes to this bug.