Bug 772679

Summary:	QpidRAMessage incorrectly calls getSession to determine if underlying JMS session has been closed
Product:	Red Hat Enterprise MRG	Reporter:	Martin Vecera <mvecera>
Component:	qpid-java	Assignee:	messaging-bugs <messaging-bugs>
Status:	CLOSED DUPLICATE	QA Contact:	MRG Quality Engineering <mrgqe-bugs>
Severity:	unspecified	Docs Contact:
Priority:	high
Version:	2.0	CC:	wprice
Target Milestone:	2.2
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
URL:	https://issues.apache.org/jira/browse/QPID-3731
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-01-11 16:01:12 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

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 ***