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

XAResourceRecovery has wrong conceptual model

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: 4.3.0.GA
    • Fix Version/s: 4.4.0.GA
    • Component/s: Recovery
    • Labels:
      None

      Description

      XAResourceRecovery is based on an iterator model:
      public boolean initialise (String p) throws SQLException;
      public boolean hasMoreResources ();
      public XAResource getXAResource () throws SQLException;

      However, XAResourceRecovery instances are created once at recovery manager startup. This means there is only ever one recovery pass that actually returns resources. On later passes the iterator is exhausted and returns nothing. This is the wrong model. XAResources are not individual entities in need of recovery. Rather they are entities from which a list of Xids in need of recovery can be retrieved. If the configuration of the underlying system does not change, the set of XAResources should remain constant over time, being one per known resource manager. On each recovery pass the list of in-doubt Xids returned by a recover scan on that resource many change.

      The model should change to follow that used by JCA ResourceAdapter:

      public XAResource[] getXAResources()

      The recovery pass would then obtain and walk the array rather than using the current iterator model.

      Additionally the exception types should be more generic. SQLException assumes the resource manager is a db, which may not be the case.

      This will also impact the recovery examples, which implement this interface.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  jhalliday Jonathan Halliday
                  Reporter:
                  jhalliday Jonathan Halliday
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: