Bug 773348

Summary:	QpidRAMessage incorrectly calls getSession to determine if underlying JMS session has been closed.
Product:	Red Hat Enterprise MRG	Reporter:	Ted Ross <tross>
Component:	qpid-jca	Assignee:	Weston M. Price <wprice>
Status:	CLOSED CURRENTRELEASE	QA Contact:	MRG Quality Engineering <mrgqe-bugs>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	2.0	CC:	cdewolf, iboverma, mvecera, tross
Target Milestone:	2.1.2
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	qpid-jca-0.14-3	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-12-07 17:42:45 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 Ted Ross 2012-01-11 15:35:50 UTC

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.

ref: https://issues.apache.org/jira/browse/QPID-3731

Comment 1 Ted Ross 2012-01-11 16:00:21 UTC

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 3 Martin Vecera 2012-02-09 09:44:17 UTC

*** Bug 772679 has been marked as a duplicate of this bug. ***

Comment 4 Martin Vecera 2012-02-09 13:02:27 UTC

Verified with JCA adapter 0.14.