[GERONIMO-4461] Improve exception during transaction manager one phase commit

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.1.4, 2.2
• Fix Version/s: 2.1.4, 2.2
• Component/s: transaction manager
• Security Level: public (Regular issues)
• Assignee: Lin Sun
• Reporter: Lin Sun
• Created: 10/Dec/08 20:53
• Updated: 16/Jan/09 22:43
• Resolved: 16/Jan/09 22:42

Description

Currently, during transaction manager one phase commit, if there is XAException arise from XAResource.commit, the code just throws RollbackException.

We should provide a more detailed Exceptions, that is:

1. If the XAException is XAException.XA_HEURRB, we throw HeuristicRollbackException, and call XAResource to forget
2. If the XAException is XAException.XA_HEURMIX, we throw HeuristicMixedException, and call XAResource to forget
3. If the XAException is XAException.XA_HEURCOM, we don't need to inform the transaction originator, but we want to call XAResource to forget.
4. Other XAException, throw RollbackException, same as the current code.

Thoughts?

Lin