Uploaded image for project: 'JBoss Transaction Manager'
  1. JBoss Transaction Manager
  2. JBTM-2891

Default dependency on org.jboss.xts could be added to WFLY deployements when deployment uses XTS interfaces

    Details

    • Type: Enhancement
    • Status: Open (View Workflow)
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 4.2
    • Fix Version/s: None
    • Component/s: XTS
    • Labels:
      None

      Description

      WildFly currently adds default dependency for org.jboss.xts module when deployment defines some of the compensations annotation. Provided by XTSDependenciesDeploymentProcessor see https://github.com/wildfly/wildfly/blob/master/xts/src/main/java/org/jboss/as/xts/XTSDependenciesDeploymentProcessor.java#L58 (in fact XTS dependency is needed only when compensating framework works in distributed manner but it's absolutely ok by my point of view to add the dependency as it's currently done).

      But if deployment uses just XTS interfaces and defines WS handlers in descriptor then user is forced to apply the dependency on its own.

      E.g. eap quickstarts does so

      https://github.com/wildfly/quickstart/blob/11.x/wsat-simple/src/main/java/org/jboss/as/quickstarts/wsat/simple/RestaurantParticipant.java#L33

      It implements Durable2PCParticipant and the XTS transaction context processing is done by specifying the xts handler in descriptor (https://github.com/wildfly/quickstart/blob/11.x/wsat-simple/src/main/webapp/WEB-INF/classes/context-handlers.xml#L24).

      It would be good to add implicit dependency in such cases for user does not need to provide it on his own.

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                ochaloup Ondrej Chaloupka
                Reporter:
                ochaloup Ondrej Chaloupka
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated: