Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-4435

Reduce the presence and configuration of multiple redundant transaction interceptors used for @Transactional

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0.3
    • Component/s: Configuration
    • Labels:
      None
    • Similar issues:
      KULRICE-6336Fix issue of multiple datasources and transaction managers being loaded and configured via CommonSpringBeans.xml
      KULRICE-13431Implement a unit test that tests transactions across multiple datasources
      KULRICE-9557Reduce default scope of transactions within KRAD
      KULRICE-8839KIM services need to be @Transactional
      KULRICE-9478KRAD Controllers are always transactional
      KULRICE-4740Eliminate redundant configuration
      KULRICE-10083Non-rollback transaction generates FATAL log message
      KULRICE-13843Create working transactional document POC
      KULRICE-10896KNS transactional documents are unable to mask their fields
      KULRICE-14087Several (hundreds) of transactions being created for a single request

      Description

      Because a lot of the Rice spring files contain <tx:annotation-driven />, based on some research that Peter did it appears that services are getting wrapped multiple times unecessarily in a transaction interceptor. Additionally, as a result of this, not all of the services are using the special KualiTransactionInterceptor which has some additional debugging implemented in it.

      Also, the current configuration in Spring that uses the DefaultAdvisorAutoProxyCreator is in the KNSSpringBeans.xml and should really be in CommonSpringBeans.xml.

        Activity

        Hide
        Eric Westfall added a comment -

        This has been fixed by removing all of the tx:annotation-driven beans and moving the DefaultAdvisorAutoProxyCreator and related beans to CommonSpringBeans.xml

        Show
        Eric Westfall added a comment - This has been fixed by removing all of the tx:annotation-driven beans and moving the DefaultAdvisorAutoProxyCreator and related beans to CommonSpringBeans.xml

          People

          • Assignee:
            Eric Westfall
            Reporter:
            Eric Westfall
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Structure Helper Panel