Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Idea to support execution listeners #27

Closed
ingorichtsmeier opened this issue Sep 8, 2022 · 5 comments · Fixed by #280
Closed

Idea to support execution listeners #27

ingorichtsmeier opened this issue Sep 8, 2022 · 5 comments · Fixed by #280

Comments

@ingorichtsmeier
Copy link
Contributor

  • Add additional headers to the Camunda 8 service task with class names for the listener implementation
  • Invoke the start listeners before the delegate implementation
  • Invoke the end listeners after the delegate implementation

Assumption: listeners have to be "clean" as well, e.g. they should not invoke execution.getProcessEngineServices().

The transaction behavior is handled in the task worker as all methods are called in a given sequence and any exception is catched in the adapter.

@jonathanlukas
Copy link
Collaborator

I like this idea. We could add this behaviour to our converter as well as to the adapter.

@jonathanlukas jonathanlukas added the enhancement New feature or request label Sep 30, 2022
@jonathanlukas
Copy link
Collaborator

Scope: Only the migration tool will support this.

@jonathanlukas
Copy link
Collaborator

@jangalinski tagging you here to match your PR from the hackday with the issue

@jangalinski
Copy link
Contributor

Thanks. I will do some cleaning up first

@jangalinski
Copy link
Contributor

see #280

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants