Seam 2
  1. Seam 2
  2. JBSEAM-4391

Need to add transaction propagation types to @Transactional

    Details

    • Type: Feature Request Feature Request
    • Status: Open Open (View Workflow)
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.2.0.GA
    • Fix Version/s: None
    • Component/s: Core
    • Labels:
      None
    • Environment:
      N/A
    • Estimated Difficulty:
      High
    • Similar Issues:
      Show 10 results 

      Description

      The current tx propagation types for use with @Transactional are the following:

      REQUIRED, MANDATORY, SUPPORTS, NEVER;

      Here is a comment from the TransactionalPropagationType enum class:

      • Transaction propagation strategies for Seam JavaBean components. Note that unlike EJB3 components, there are no strategies for suspending transactions.

      Please add REQUIRES_NEW and NOT_SUPPORTED. I am unable to solve my functional requirement of calling a webservice which inserts into remote db (which does not require a tx) and inserting into multiple local db tables (requires tx) using JavaBean components and Seam @Transactional. The requirement is that even if the webservice call fails, the local db inserts should continue and succeed (i.e. no atomicity, no rollback).

      There is an action method in my JavaBean Seam component that is invoked from a commandButton in JSF page. Then there are two private methods that are invoked from that public method. One requires tx support, the other does not. With EJB3, I would simply use NOT_SUPPORTED for one and REQUIRES_NEW for the other. Now using JavaBeans and @Transactional, I cannot solve my problem in terms of tx support requirements. @Transactional only works on public methods.

      I tried recalling the component's private method (now refactored to public with @Transactional) using Component.getInstance("foo"); but that did not work.

      Spring has robust tx support like EJB 3. Seam needs to be ramped up in terms of tx support for JavaBean components.

      References:

      http://www.seamframework.org/Community/TransactionalPropagationTypes
      http://www.seamframework.org/Community/TransactionalOnPrivateMethod

      1. jbseam-requires-new-1.diff
        9 kB
        Stuart Douglas
      2. jbseam-requires-new-2.diff
        9 kB
        Stuart Douglas
      3. jbseam-requires-new-3.diff
        23 kB
        Stuart Douglas

        Issue Links

          Activity

          Hide
          Stuart Douglas
          added a comment -

          I have attached a patch that implements basic REQUIRES_NEW functionality.

          Show
          Stuart Douglas
          added a comment - I have attached a patch that implements basic REQUIRES_NEW functionality.
          Hide
          Stuart Douglas
          added a comment -

          Updated patch with a minor fix

          Show
          Stuart Douglas
          added a comment - Updated patch with a minor fix
          Hide
          Stuart Douglas
          added a comment -

          I have attached a patch that prevents the propegation of the SMPC across transaction boundaries and includes integration tests.

          Show
          Stuart Douglas
          added a comment - I have attached a patch that prevents the propegation of the SMPC across transaction boundaries and includes integration tests.
          Hide
          Arbi Sookazian
          added a comment -

          Cool!

          Show
          Arbi Sookazian
          added a comment - Cool!
          Hide
          Andrei Ivanov
          added a comment -

          Any chance for someone looking at this?

          Show
          Andrei Ivanov
          added a comment - Any chance for someone looking at this?

            People

            • Assignee:
              Unassigned
              Reporter:
              Arbi Sookazian
            • Votes:
              5 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated: