You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Regis Xu (JIRA)" <ji...@apache.org> on 2009/05/05 10:05:30 UTC

[jira] Created: (HARMONY-6195) [classlib][text] - improve performance for SimpleDateFormat.format/parse

[classlib][text] - improve performance for SimpleDateFormat.format/parse
------------------------------------------------------------------------

                 Key: HARMONY-6195
                 URL: https://issues.apache.org/jira/browse/HARMONY-6195
             Project: Harmony
          Issue Type: Improvement
          Components: Classlib
    Affects Versions: 5.0M9
            Reporter: Regis Xu
         Attachments: DateFormatBench.java

Harmony delegate SimpleDateFormat.format/parse to icu, but the speed is slower than icu. The only extra wok is setTimeZone before delegating. Since time zone is rarely changed, we can only set it when changed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-6195) [classlib][text] - improve performance for SimpleDateFormat.format/parse

Posted by "Regis Xu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6195?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Regis Xu updated HARMONY-6195:
------------------------------

    Attachment: HARMONY-6195.diff

 add new field tzId to avoid setTimeZone every time

> [classlib][text] - improve performance for SimpleDateFormat.format/parse
> ------------------------------------------------------------------------
>
>                 Key: HARMONY-6195
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6195
>             Project: Harmony
>          Issue Type: Improvement
>          Components: Classlib
>    Affects Versions: 5.0M9
>            Reporter: Regis Xu
>         Attachments: DateFormatBench.java, HARMONY-6195.diff
>
>
> Harmony delegate SimpleDateFormat.format/parse to icu, but the speed is slower than icu. The only extra wok is setTimeZone before delegating. Since time zone is rarely changed, we can only set it when changed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (HARMONY-6195) [classlib][text] - improve performance for SimpleDateFormat.format/parse

Posted by "Regis Xu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6195?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Regis Xu reassigned HARMONY-6195:
---------------------------------

    Assignee: Regis Xu

> [classlib][text] - improve performance for SimpleDateFormat.format/parse
> ------------------------------------------------------------------------
>
>                 Key: HARMONY-6195
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6195
>             Project: Harmony
>          Issue Type: Improvement
>          Components: Classlib
>    Affects Versions: 5.0M9
>            Reporter: Regis Xu
>            Assignee: Regis Xu
>         Attachments: DateFormatBench.java, HARMONY-6195.diff
>
>
> Harmony delegate SimpleDateFormat.format/parse to icu, but the speed is slower than icu. The only extra wok is setTimeZone before delegating. Since time zone is rarely changed, we can only set it when changed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-6195) [classlib][text] - improve performance for SimpleDateFormat.format/parse

Posted by "Regis Xu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6195?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Regis Xu updated HARMONY-6195:
------------------------------

    Attachment: DateFormatBench.java

a micro benchmark for this.

> [classlib][text] - improve performance for SimpleDateFormat.format/parse
> ------------------------------------------------------------------------
>
>                 Key: HARMONY-6195
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6195
>             Project: Harmony
>          Issue Type: Improvement
>          Components: Classlib
>    Affects Versions: 5.0M9
>            Reporter: Regis Xu
>         Attachments: DateFormatBench.java
>
>
> Harmony delegate SimpleDateFormat.format/parse to icu, but the speed is slower than icu. The only extra wok is setTimeZone before delegating. Since time zone is rarely changed, we can only set it when changed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HARMONY-6195) [classlib][text] - improve performance for SimpleDateFormat.format/parse

Posted by "Regis Xu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HARMONY-6195?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12705948#action_12705948 ] 

Regis Xu commented on HARMONY-6195:
-----------------------------------

I ran the micro benchmark 5 times before and after the patch respectively

before:
java.text.SimpleDateFormat.format: 4602    4363    4614    4668    4750
java.text.SimpleDateFormat.parse: 4664    5899    5016    5973    6128
com.ibm.icu.text.SimpleDateFormat.format: 2308    2504    2315    2155    2492
com.ibm.icu.text.SimpleDateFormat.parse: 3566    2960    2653    2564    3605

after:
java.text.SimpleDateFormat.format: 3262    3355    3203    3311    3502
java.text.SimpleDateFormat.parse: 3841    3786    3185    3137    4107
com.ibm.icu.text.SimpleDateFormat.format: 2317    2146    2284    2372    2252
com.ibm.icu.text.SimpleDateFormat.parse: 3446    3506    3830    4019    3744

java.text.SimpleDateFormat.format is still slower 50% than com.ibm.icu.text.SimpleDateFormat.format.
Any suggestion/comments are welcome.




> [classlib][text] - improve performance for SimpleDateFormat.format/parse
> ------------------------------------------------------------------------
>
>                 Key: HARMONY-6195
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6195
>             Project: Harmony
>          Issue Type: Improvement
>          Components: Classlib
>    Affects Versions: 5.0M9
>            Reporter: Regis Xu
>         Attachments: DateFormatBench.java, HARMONY-6195.diff
>
>
> Harmony delegate SimpleDateFormat.format/parse to icu, but the speed is slower than icu. The only extra wok is setTimeZone before delegating. Since time zone is rarely changed, we can only set it when changed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (HARMONY-6195) [classlib][text] - improve performance for SimpleDateFormat.format/parse

Posted by "Regis Xu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6195?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Regis Xu closed HARMONY-6195.
-----------------------------


> [classlib][text] - improve performance for SimpleDateFormat.format/parse
> ------------------------------------------------------------------------
>
>                 Key: HARMONY-6195
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6195
>             Project: Harmony
>          Issue Type: Improvement
>          Components: Classlib
>    Affects Versions: 5.0M9
>            Reporter: Regis Xu
>            Assignee: Regis Xu
>             Fix For: 5.0M10
>
>         Attachments: DateFormatBench.java, HARMONY-6195.diff
>
>
> Harmony delegate SimpleDateFormat.format/parse to icu, but the speed is slower than icu. The only extra wok is setTimeZone before delegating. Since time zone is rarely changed, we can only set it when changed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (HARMONY-6195) [classlib][text] - improve performance for SimpleDateFormat.format/parse

Posted by "Regis Xu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6195?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Regis Xu resolved HARMONY-6195.
-------------------------------

       Resolution: Fixed
    Fix Version/s: 5.0M10

Patch applied at r772870.

> [classlib][text] - improve performance for SimpleDateFormat.format/parse
> ------------------------------------------------------------------------
>
>                 Key: HARMONY-6195
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6195
>             Project: Harmony
>          Issue Type: Improvement
>          Components: Classlib
>    Affects Versions: 5.0M9
>            Reporter: Regis Xu
>            Assignee: Regis Xu
>             Fix For: 5.0M10
>
>         Attachments: DateFormatBench.java, HARMONY-6195.diff
>
>
> Harmony delegate SimpleDateFormat.format/parse to icu, but the speed is slower than icu. The only extra wok is setTimeZone before delegating. Since time zone is rarely changed, we can only set it when changed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.