[
http://jira.qos.ch/browse/LBCLASSIC-36?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=11181#action_11181
]
Ralph Goers commented on LBCLASSIC-36:
--------------------------------------
There are several problems with your updated tests.
1. There is no reason to synchronize in the Joda test. DateTimeFormatter should
be declared external to the method and the synchronization should be removed
since DateTimeFormatter is thread safe.
2. You are not testing thread contention.
I have modified the test to correct this and am adding it here. I am not sure
that it adequately emulates logging though, since there is no other work
occurring besides formatting the time.
> 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
logback-dev@qos.ch
http://qos.ch/mailman/listinfo/logback-dev
