httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Graham Leggett <minf...@sharp.fm>
Subject Re: r->connection->aborted in mod_disk_cache and mod_mem_cache
Date Mon, 08 Mar 2010 15:13:13 GMT
On 08 Mar 2010, at 4:37 PM, Kevac Marko wrote:

> Hello. Why this code is presented in disk cache, but not in mem cache?
>
>        if (r->connection->aborted || r->no_cache) {
>            ap_log_error(APLOG_MARK, APLOG_INFO, 0, r->server,
>                         "disk_cache: Discarding body for URL %s "
>                         "because connection has been aborted.",
>                         h->cache_obj->key);
>            /* Remove the intermediate cache file and return non- 
> APR_SUCCESS */
>            file_cache_errorcleanup(dobj, r);
>            return APR_EGENERAL;
>        }

The mem cache hasn't received the same level of attention is the disk  
cache did, and due to the OS typically backing the disk with a memory  
cache anyway, there wasn't much that mem cache did that disk cache  
didn't already do anyway. As a result, mem cache was removed from what  
is to become v2.4.

That said, if it fixes a bug in v2.2, it can definitely be proposed  
for inclusion.

Regards,
Graham
--


Mime
View raw message