Thank you Kris,
I've submitted the patch:
https://jira.jboss.org/jira/browse/JBRULES-1887
No need to hurry with this.
I haven't tested it. For some reason I can't run my application with locally build drools jars.
Keep up the good work!
Michal



On Fri, Dec 5, 2008 at 3:37 PM, Kris Verlaenen <kris.verlaenen@cs.kuleuven.be> wrote:
Michal,
 
True, thanks for the help.  I suggest you just try to change the code locally so that it fits your requirements, and bundle your comments (and a patch would be nice :)) in a JIRA.  I'll take a look and will try to incorporate those changes in trunk asap !
 
Kris
 
----- Original Message -----
From: Michal Bali
Sent: Wednesday, December 03, 2008 10:48 PM
Subject: [rules-dev] drools-process-enterprise - JPAPersisterManager shouldtake EntityManagerFactory as an input parameter

Hi Kris,

The JPAPersisterManager is currently creating a new EntityManagerFactory.

It should probably also take an EntityManagerFactory as an input. This is necessary if you already have an EntityManagerFactory in your application you want to reuse it. Two EntityManagerFactories mean two transactional resources that have to be managed by a distributed transaction and that is unnecessary.

If you want I should be able to provide the patch. It should be a simple constructor change :).

I hope I am not starting to bother you with my issues, I might have a few more :)

Thank you.
Best Regards,
Michal

_______________________________________________
rules-dev mailing list
rules-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/rules-dev

_______________________________________________
rules-dev mailing list
rules-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/rules-dev