Thanks for the feedback, Matthieu.

> > I played around a bit with this and for a start just changed the
> > special handling of in-memory processes in
> > org.apache.ode.bpel.engine.PartnerLinkPartnerRoleImpl.invokeIL(...):
> > [...]
> > ... and it worked fine in our use case and it didn't make any notable
> > difference in your unit tests ("buildr test" in trunk). Is it really that 
> > simple?
> > Then I could provide a patch to make this configurable (globally in
> > OdeConfigProperties?
> > Or per process in deploy.xml? Maybe better, because performance is
> > probably better with synchronous execution so it would make sense to enable 
> > this only
> > for processes that actually contain <flow>s).
> >
> > Or are there any pitfalls with this approach?
> 
> I don't think so, the code between the two methods is very similar
> anyway if you don't consider the transaction suspend in the in-mem case.
> 
> I would also favor per-process configuration for this.

OK, I'll prepare the patch for this (next week, probably ;-).

Cheers,
Jürgen.

Reply via email to