Issue Details (XML | Word | Printable)

Key: XADISK-115
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Critical Critical
Assignee: Nitin Verma
Reporter: Simon Dierl
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
xadisk

JBoss 7.1 expects #run() and #release to be explicitly declared

Created: 15/May/12 11:30 AM   Updated: 27/Aug/13 07:22 AM   Resolved: 09/Aug/13 02:32 PM
Component/s: None
Affects Version/s: 1.2.1
Fix Version/s: 1.2.2

Time Tracking:
Not Specified

File Attachments: 1. Text File xadisk-run-release.patch (2 kB) 15/May/12 11:30 AM - Simon Dierl

Environment:

JBoss 7.1.1


Tags:
Participants: Nitin Verma and Simon Dierl


 Description  « Hide

The #run() and #release() methods required by javax.resource.spi.work.Work are expected by JBoss 7.1 to be overridden in the specific subclasses of Work used in an RA. Unfortunately, XADisk only declared these methods in abstract superclasses; the concrete implementations only inherit these.

While this is perfectly fine with respect to the Spec, JBoss will refuse to load XADisk unless the subclasses override the methods from the abstract Work classes (overriding with super.run() / super.release()) is okay).

While JBoss is quite certainly wrong here, XADisk will not load on JBoss 7.1; the attached patch fixes that (without changing behavior).



No work has yet been logged on this issue.