search

plexiti / camunda-grails-plugin

Integrates Camunda BPM with the Grails framework.
http://grails.org/plugin/camunda
Apache License 2.0
18 stars 8 forks source link

fix(setup): do not register process engine with JMX container #17

Closed meyerdan closed 10 years ago

meyerdan commented 10 years ago

this transforms the setup into a purely embedded setup which seems to be a better fit for this environment.

related to #16

meyerdan commented 10 years ago

I did not test my changes, so merge with care :)

martinschimak commented 10 years ago

Hi Daniel - many thx. I actually already tried exactly that yesterday evening, but I think I forgot to remove the SpringServletProcessApplication... a first quick test shows: this seems to fix the issue now!

The original reason why I used the "managed" version was that I wanted to support a cockpit overlay in a coming version of the plugin - for which I remember to need that. However, this is not yet supported by the plugin at the moment anyway - so I can later add that and make sure that the migration plugin will continue to work... my suspicion is that this plugin bootstraps a second, separate application context.

I will test your pull request more thorougly later and then assumedly merge it.

Manas101 commented 10 years ago

Hi, the fix works for me. Thank you for your quick reply!