Bug 738316

Summary: Qpid JCA Adapter -- QpidRALocalTransaction incorrectly evaluates session.getTransacted() flag in commit
Product: Red Hat Enterprise MRG Reporter: Weston M. Price <wprice>
Component: qpid-jcaAssignee: Weston M. Price <wprice>
Status: CLOSED CURRENTRELEASE QA Contact: MRG Quality Engineering <mrgqe-bugs>
Severity: urgent Docs Contact:
Priority: high    
Version: 2.0CC: cdewolf, iboverma, jross, tross
Target Milestone: 2.0.4   
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: qpid-java-jca-0.10-10 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-02-24 16:42:47 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 737880    

Description Weston M. Price 2011-09-14 14:22:27 UTC 
In the QpidRALocalTransaction class, in the commit method, prior to committing we evaluate the following conditions:

 if (_mc.getSession() != null && _mc.getSession().getTransacted())

While the first condition is correct, the second is not. Rarely, if ever will this flag be true in a JCA context. Also, this flag should not be used to evaluate whether or not to commit as commit is actually being called by the transaction manager. 


This issue is the result of QE testing, Bug 737880.
Comment 1 Weston M. Price 2011-09-16 15:36:03 UTC 
Provided in latest build to release engineering.
Comment 2 Andrew Stitcher 2011-09-22 13:30:09 UTC 
This has been fixed in package qpid-java-jca-0.10-10