hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Moore, Jonathan (CIM)" <Jonathan_Mo...@Comcast.com>
Subject RE: [jira] [Commented] (HTTPCLIENT-1116) ResponseCachingPolicy uses integers for sizes
Date Mon, 15 Aug 2011 11:26:18 GMT
Another option would be:
(1) change internal usage to long
(2) *add* getter/setter using long to CacheConfig (don't replace the int versions)

This preserves existing functionality but allows for the > 2GB use case.

From: Bill Speirs [bill.speirs@gmail.com]
Sent: Sunday, August 14, 2011 9:44 PM
To: HttpComponents Project
Subject: Re: [jira] [Commented] (HTTPCLIENT-1116) ResponseCachingPolicy uses integers for

Why wouldn't you throw an exception? Regardless, people are going to have to
change their code when dealing with sizes > MAX. With the exception you know
exactly what the issue is. With returning MAX, it could take the user much
longer to track down the issue.

My $0.02...


On Aug 14, 2011 5:30 PM, "Oleg Kalnichevski (JIRA)" <jira@apache.org> wrote:
> [
> Oleg Kalnichevski commented on HTTPCLIENT-1116:
> -----------------------------------------------
> As far as I can tell the second option should be good enough.
> Oleg
>> ResponseCachingPolicy uses integers for sizes
>> ---------------------------------------------
>> Key: HTTPCLIENT-1116
>> URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1116
>> Project: HttpComponents HttpClient
>> Issue Type: Bug
>> Components: Cache
>> Affects Versions: 4.1.1, 4.2 Alpha1
>> Reporter: Greg Bowyer
>> Labels: cache, policy
>> Attachments: 0001-Changed-the-ResponseCachingPolicy-to-use-longs.patch
>> ResponseCachingPolicy currently uses integers for interpreting the size
of Content-Length, as well internally.
>> This causes issues in attempting to use the module for caching entities
that are over 2GB in size, the module does not fail gracefully, but throws a
>> I have a patch that fixes this, by promoting the int -> long, which
should allow for larger entities to be cached, it also updates the public
facing API where possible, I don't think that the promotion should break
compatibility massively
>> The changes can also be seen here:
> --
> This message is automatically generated by JIRA.
> For more information on JIRA, see: http://www.atlassian.com/software/jira
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
> For additional commands, e-mail: dev-help@hc.apache.org

To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org

View raw message