[OFBIZ-4282] TransactionUtil performance optimisations

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Trunk
• Fix Version/s: Trunk
• Component/s: framework
• Labels: PERFORMANCE
• Assignee: Jacques Le Roux
• Reporter: Philippe Mouawad
• Created: 14/May/11 08:54
• Updated: 01/Jan/12 11:29
• Resolved: 01/Jan/12 11:29

Description

Hello,
Reviewing TransactionUtil code, I have seen 2 problems:

• internalBegin is uselessly synchronized , since it is a static method it is a very big useless Contention Point since not unthread safe instance variable is used
• debugResources is true which creates a DebugXAResource (that creates an Exception) , it should be false and made an option for debuging

These 2 modifications have been in our production for a while and we noticed CPU reduction and no more contention on TransactionUtil#begin

Regards
Philippe Mouawad
http://www.ubik-ingenierie.com

Sub-Tasks

• Remove Synchronized in internalBegin: Closed
• Set TransactionUtil#debugResources default value to false and make it configurable: Closed
• ERROR in log file when starting OFBiz, debug-xa-resources is commented in entityengine.xml: Closed