I've attached a Yourkit snapshot of the process.

I agree with Ben that it's probably opening up multiple transactions since we 
removed the annotation from LogicService.  It was the high level place where 
all the processing started.

Thanks,
Steve


From: [email protected] [mailto:[email protected]] On Behalf Of Darius Jazayeri
Sent: Friday, August 12, 2011 2:25 PM
To: [email protected]
Subject: Re: [OPENMRS-DEV] Logic Transactional Annotations

Are you able to run the Yourkit profiler against your running app?

I'd be happy to help interpret a yourkit snapshot.

-Darius
On Fri, Aug 12, 2011 at 10:48 AM, McKee, Steven Jay 
<[email protected]<mailto:[email protected]>> wrote:
I removed the transactional annotation from the LogicService interface, but I'm 
seeing tremendous slowdowns in our rule processing.  Rule processing to create 
one of our forms usually takes around 7-10 seconds to complete.  After the 
logic change, it's taking around 30-70 seconds.  I tried applying the read-only 
transactional annotation to the logic DAO's, but that didn't make a difference 
either.  I also modified our Arden parser to make our rules use the 
LogicContext object instead of LogicService (as suggested earlier in this 
thread) within logic rules, but that didn't make a difference either.

I apologize if this message gets posted to the developers list multiple times.  
I have issues about every time I try to post a message, and no one has been 
able to figure out why.

Steve

From: [email protected]<mailto:[email protected]> 
[mailto:[email protected]]<mailto:[mailto:[email protected]]> On Behalf Of Darius 
Jazayeri
Sent: Tuesday, August 02, 2011 2:43 PM
To: 
[email protected]<mailto:[email protected]>
Subject: Re: [OPENMRS-DEV] Logic Transactional Annotations

Steve, can you attach this as a patch against 1.7.x on the relevant ticket, for 
code review and inclusion?

(If you can comment on how much testing you've done after making this fix, 
that'd be helpful. Surely it'll be 1000% more than the testing anyone else has 
done on logic.)

-Darius

_________________________________________

To unsubscribe from OpenMRS Developers' mailing list, send an e-mail to 
[email protected] with "SIGNOFF openmrs-devel-l" in the  body (not 
the subject) of your e-mail.

[mailto:[email protected]?body=SIGNOFF%20openmrs-devel-l]

Reply via email to