[ 
https://issues.apache.org/jira/browse/HTTPCORE-79?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12505595
 ] 

Roland Weber commented on HTTPCORE-79:
--------------------------------------

Hello Daniel,

target _environment_ is...
a) client HTTP application, few threads, few operations -> current 
implementation is fine
b) server application, few threads, many operations -> full per-thread caching 
is fine
c) server application, many threads, few operations -> current implementation 
not so good, full caching not so good either
d) ....

Within HttpCore, only formatting should be required. Date parsing is for 
cookies. So maybe reduce DateUtils to formatting only, based on FastDateFormat. 
Date parsing goes to HttpCookie and gets an interface for substituting 
different implementations.

cheers,
  Roland

 

> DateUtils should cache SimpleDateFormat
> ---------------------------------------
>
>                 Key: HTTPCORE-79
>                 URL: https://issues.apache.org/jira/browse/HTTPCORE-79
>             Project: HttpComponents Core
>          Issue Type: Improvement
>          Components: HttpCore
>    Affects Versions: 4.0-alpha5
>            Reporter: Daniel Müller
>            Priority: Minor
>             Fix For: 4.0-beta1
>
>         Attachments: org.apache.http.util.patch
>
>
> DateUtils create a SimpleDateFormat for each invocation of #formatDate and 
> #parseDate. This can be optimized if SimpleDateFormat instances are cached. 
> Since SimpleDateFormat is not threadsafe, the cache must be threadlocal.

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


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

Reply via email to