[ 
https://issues.apache.org/jira/browse/TAPESTRY-1229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12466274
 ] 

Massimo Lusetti commented on TAPESTRY-1229:
-------------------------------------------

Kudos for this catch, i think i came accross this once or twice without 
recognizing it, thanks!

> AssetService not thread safe
> ----------------------------
>
>                 Key: TAPESTRY-1229
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1229
>             Project: Tapestry
>          Issue Type: Bug
>          Components: Core, Framework
>    Affects Versions: 4.1.1
>            Reporter: Greg Woolsey
>            Priority: Blocker
>             Fix For: 4.1.2
>
>
> AssetService declares this field:
>     static final DateFormat CACHED_FORMAT = new SimpleDateFormat("EEE, d MMM 
> yyyy HH:mm:ss Z", Locale.ENGLISH);
> However, DateFormat objects are explicitly NOT thread safe.  We've 
> encountered this in our in-house concurrency testing.  We'll have to patch 
> our local 4.1.1 to fix this before deploying our application.
> a quick search did not find any other references to SimpleDateFormat that 
> look like they could have the same issue.
> A simple CommonsPool object pool of date format objects would fix the problem.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to