I don't see an attachment here. Perhaps it's too big? Can you attach it to a ticket?
-Darius On Fri, Aug 12, 2011 at 12:15 PM, McKee, Steven Jay <[email protected]>wrote: > 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]> > 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]] *On Behalf Of *Darius > Jazayeri > *Sent:* Tuesday, August 02, 2011 2:43 PM > *To:* [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**** > > **** > ------------------------------ > Click here to > unsubscribe<[email protected]?body=SIGNOFF%20openmrs-devel-l>from > OpenMRS Developers' mailing list > _________________________________________ 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]
