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 < [EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> > *To:* Rules Dev List <[email protected]> > *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 > [email protected] > https://lists.jboss.org/mailman/listinfo/rules-dev > > > _______________________________________________ > rules-dev mailing list > [email protected] > https://lists.jboss.org/mailman/listinfo/rules-dev > >
_______________________________________________ rules-dev mailing list [email protected] https://lists.jboss.org/mailman/listinfo/rules-dev
