[OFBIZ-4201] DateFormat.getDateTimeInstance() is very expensive, we can cache it to improve performance

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Implemented
• Affects Version/s: Release Branch 10.04
• Component/s: framework
• Sprint: Bug Crush Event - 21/2/2015
• Assignee: Jacques Le Roux
• Reporter: Xiaoming Shi
• Created: 25/Feb/11 02:09
• Updated: 13/Dec/18 15:06
• Resolved: 13/Dec/18 15:06

Description

In the files:

./apache-ofbiz-10.04/framework/base/src/org/ofbiz/base/util/UtilDateTime.java line:702
./apache-ofbiz-10.04/framework/base/src/org/ofbiz/base/util/UtilFormatOut.java line:314
./apache-ofbiz-10.04/framework/webapp/src/org/ofbiz/webapp/taglib/FormatTag.java line: 78

DateFormat.getDateTimeInstance() is called for every function call of "toGmtTimestampString" or "formatDate". We can cache the value by adding a static class field and improve performance.

This is similar to the Apache Bug https://issues.apache.org/bugzilla/show_bug.cgi?id=48778