OFBiz
  1. OFBiz
  2. OFBIZ-4201

DateFormat.getDateTimeInstance() is very expensive, we can cache it to improve performance

    Details

    • Type: Improvement Improvement
    • Status: Reopened
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: Release Branch 10.04
    • Fix Version/s: Trunk
    • Component/s: framework
    • Labels:
      None

      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

        Activity

        Hide
        Jacques Le Roux added a comment -

        Thanks for report Shi,

        It's done in trunk at r1165076

        Show
        Jacques Le Roux added a comment - Thanks for report Shi, It's done in trunk at r1165076
        Show
        Jacques Le Roux added a comment - We will certainly use http://commons.apache.org/lang/api-2.5/org/apache/commons/lang/time/FastDateFormat.html rather

          People

          • Assignee:
            Jacques Le Roux
            Reporter:
            Xiaoming Shi
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development