Activiti

Add Spring support in a JTA and appserver environment

Details

  • Type: Improvement Improvement
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: Sooner
  • Component/s: Engine
  • Labels:
    None
  • Number of attachments :
    0

Description

Potentially this involves: Finishing transaction manager integration for Spring so that ACtiviti TX interceptors are respected properly and REQUIRES_NEW is respected / enforced where necessary.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Tom Baeyens added a comment -

I believe these were intended for future releases. I assigned them to beta 1 release so that we can discuss them in context of the next release

Show
Tom Baeyens added a comment - I believe these were intended for future releases. I assigned them to beta 1 release so that we can discuss them in context of the next release
Tom Baeyens made changes -
Field Original Value New Value
Fix Version/s 5.0.beta1 [ 16575 ]
Fix Version/s 5.0.alpha4 [ 16574 ]
Tom Baeyens made changes -
Affects Version/s 5.0.beta2 [ 16576 ]
Affects Version/s 5.0.alpha4 [ 16574 ]
Tom Baeyens made changes -
Affects Version/s 5.0.beta2 [ 16576 ]
Fix Version/s 5.0.beta2 [ 16576 ]
Fix Version/s 5.0.beta1 [ 16575 ]
Tom Baeyens made changes -
Summary Finish transaction manager integration for Spring Make Spring integration work in a JTA and appserver environment
Assignee Dave Syer [ david_syer ] Tom Baeyens [ tombaeyens ]
Description Finish transaction manager integration for Spring so that ACtiviti TX interceptors are respected properly and REQUIRES_NEW is respected / enforced where necessary. Potentially this involves: Finishing transaction manager integration for Spring so that ACtiviti TX interceptors are respected properly and REQUIRES_NEW is respected / enforced where necessary.
Component/s Engine [ 14460 ]
Tom Baeyens made changes -
Summary Make Spring integration work in a JTA and appserver environment Add Spring support in a JTA and appserver environment
Fix Version/s 5.0.beta2 [ 16576 ]
Hide
Permalink
Tom Baeyens added a comment -

this will require that we split the command executor in 2 versions, a required and a requiresNew.
the requiresNew will have to be used in DecrementJobRetriesListener and in the DbIdGenerator.

Show
Tom Baeyens added a comment - this will require that we split the command executor in 2 versions, a required and a requiresNew. the requiresNew will have to be used in DecrementJobRetriesListener and in the DbIdGenerator.
Tom Baeyens made changes -
Fix Version/s 5.3 [ 17056 ]
Tom Baeyens made changes -
Priority Major [ 3 ] Critical [ 2 ]
Hide
Permalink
Bernd Ruecker (camunda) added a comment -

There is one related bug in the DbIdGenerator (see TODO in the getNewBlock()), I hope that will be fixed within this isse? That would be perfect, since this is a show stopper for going live in big installations.

Show
Bernd Ruecker (camunda) added a comment - There is one related bug in the DbIdGenerator (see TODO in the getNewBlock()), I hope that will be fixed within this isse? That would be perfect, since this is a show stopper for going live in big installations.
Hide
Permalink
Tom Baeyens added a comment -

Bernd, that should already be covered.
dbIdGenerator.setCommandExecutor(commandExecutorTxRequiresNew);

The only thing we don't yet have is a configuration file that shows how to configure the JTA transaction context. And QA runs that show we operate well in spring tx context and in jta tx context.

We should see progress in the qa this week. Almost there.

Show
Tom Baeyens added a comment - Bernd, that should already be covered. dbIdGenerator.setCommandExecutor(commandExecutorTxRequiresNew); The only thing we don't yet have is a configuration file that shows how to configure the JTA transaction context. And QA runs that show we operate well in spring tx context and in jta tx context. We should see progress in the qa this week. Almost there.
Tom Baeyens made changes -
Fix Version/s 5.4 [ 17124 ]
Fix Version/s 5.3 [ 17056 ]
Tom Baeyens made changes -
Priority Critical [ 2 ] Major [ 3 ]
Hide
Permalink
Tom Baeyens added a comment -

The strategy is going to be the following:

We plan to add a test suite to Alfresco test suite that reads all the tests to include from a resource file.
We'll create a profile so that the activiti engine test module will include as a resource the lists all the tests. That way, the activiti engine test classes library can just be included in the Alfresco test classpath. This should then happen each time when the activiti jar is updated in the alfresco codebase.

Frederik is going to ask the Gavin and Nick where we can include extra test dependencies. I'll investigate how we can move the generation of that resource that lists all the tests from activiti-spring module to the activiti-engine module.

Show
Tom Baeyens added a comment - The strategy is going to be the following: We plan to add a test suite to Alfresco test suite that reads all the tests to include from a resource file. We'll create a profile so that the activiti engine test module will include as a resource the lists all the tests. That way, the activiti engine test classes library can just be included in the Alfresco test classpath. This should then happen each time when the activiti jar is updated in the alfresco codebase. Frederik is going to ask the Gavin and Nick where we can include extra test dependencies. I'll investigate how we can move the generation of that resource that lists all the tests from activiti-spring module to the activiti-engine module.
Tom Baeyens made changes -
Fix Version/s Sooner [ 17208 ]
Fix Version/s 5.4 [ 17124 ]
Hide
Permalink
Bernd Ruecker (camunda) added a comment -

Outdated, please re-open if still relevant (I hope not )

Show
Bernd Ruecker (camunda) added a comment - Outdated, please re-open if still relevant (I hope not )
Bernd Ruecker (camunda) made changes -
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]

People

  • Assignee:
    Tom Baeyens
    Reporter:
    Dave Syer
Vote (3)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: