[
http://jira.qos.ch/browse/LBCLASSIC-36?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=11179#action_11179
]
Ceki Gulcu edited comment on LBCLASSIC-36 at 6/14/09 9:28 PM:
--------------------------------------------------------------
Thank you for this contribution. I have ran a modified (and simplified) version
of your tests. It looks like SimpleDateFormat and Yoda time have roughly the
same performance. with the former having the edge.
Your version of the test had Yoda time transform Date objects. However, in
logback, a "long" as obtained by System.currentMillis(), needs to be
transformed. Transforming Date objects is pretty much beside the point.
was (Author: [email protected]):
Thank you for this patch. I have ran a modified (and simplified) version of
your tests. It looks like SimpleDateFormat and Yoda time have roughly the same
performance. with the former having the edge.
> Synchronization of SimpleDateFormat object in DateConverter
> -----------------------------------------------------------
>
> Key: LBCLASSIC-36
> URL: http://jira.qos.ch/browse/LBCLASSIC-36
> Project: logback-classic
> Issue Type: Bug
> Components: Other
> Affects Versions: unspecified
> Environment: Operating System: All
> Platform: All
> Reporter: Anders Hammar
> Assignee: Ceki Gulcu
> Attachments: DateConverter_patch.java, dateformattest.zip
>
>
> According to the API documentation of SimpleDateFormat, date formats are not
> synchronized and it is recommended to create separate format instances for
> each thread.
> However, a possibly simpler (mer efficient) solution would be to synchronize
> on the SimpleDateFormat object in the convert() method to prevent two threads
> to access it concurrently.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.qos.ch/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
_______________________________________________
logback-dev mailing list
[email protected]
http://qos.ch/mailman/listinfo/logback-dev
