[ 
https://issues.apache.org/jira/browse/HTTPCLIENT-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12902389#action_12902389
 ] 

Oleg Kalnichevski commented on HTTPCLIENT-978:
----------------------------------------------

> I believe the eventual goal is to use the HttpCacheOperationException to 
> trigger a configurable number of retries on update failure, 
> so the current implementation can be reevaluated at that point if necessary.

Could HttpCacheUpdateException be a better name for such exception? It could 
also contain a piece of information (just a Boolean attribute) indicating 
whether or not the failure is fatal. The consumer of HttpCacheStorage 
implementation could choose to catch the exception and retry the operation, if 
the failure is non fatal. HttpCacheUpdateException may still extend IOException 
for the sake of simplicity. 

> A quick fix, my coworker pointed out a small bug in 
> TestHttpCacheEntrySerializers, line 86, "one.getAllHeaders()" should be 
> "two.getAllHeaders()". 

Fixed.

Cheers

Oleg


> provide an ehcache implementation for HttpCache
> -----------------------------------------------
>
>                 Key: HTTPCLIENT-978
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-978
>             Project: HttpComponents HttpClient
>          Issue Type: New Feature
>          Components: Cache
>            Reporter: Jonathan Moore
>             Fix For: 4.1 Alpha3
>
>         Attachments: ehcache_w_serialization.patch, 
> ehcache_w_serialization3.patch, httpclient-cache_ehcache-patch.patch, 
> httpclient-cache_ehcache-patch2.patch
>
>
> Provide an implementation of the HttpCache interface that stores cache 
> entries in ehcache.

-- 
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: dev-unsubscr...@hc.apache.org
For additional commands, e-mail: dev-h...@hc.apache.org

Reply via email to