[PATCH 01 of 31] Cache: fix another "stalled cache updating" alert

Kirill A. Korinskiy catap at catap.ru
Mon Jun 27 22:58:40 MSD 2011


On Mon, Jun 27, 2011 at 22:04, Kirill A. Korinskiy <catap at catap.ru> wrote:

> On Mon, Jun 27, 2011 at 21:06, Maxim Dounin <mdounin at mdounin.ru> wrote:
>
>> # HG changeset patch
>> # User Maxim Dounin <mdounin at mdounin.ru>
>> # Date 1309178032 -14400
>> # Node ID b857d44e9644325dec7d63b8561ae3b4dfa16150
>> # Parent  1c167244d2fdb064c159012c50a7ae3fd1ed254a
>> Cache: fix another "stalled cache updating" alert.
>> @@ -2991,16 +2982,19 @@ ngx_http_upstream_finalize_request(ngx_h
>>
>>
> [..]
>
>
>>  #if (NGX_HTTP_CACHE)
>>
>> -    if (u->cacheable && r->cache) {
>> -        time_t  valid;
>> -
>> -        if (rc == NGX_HTTP_BAD_GATEWAY || rc ==
>> NGX_HTTP_GATEWAY_TIME_OUT) {
>> -
>> -            valid = ngx_http_file_cache_valid(u->conf->cache_valid, rc);
>> -
>> -            if (valid) {
>> -                r->cache->valid_sec = ngx_time() + valid;
>> -                r->cache->error = rc;
>> +    if (r->cache) {
>> +
>> +        if (u->cacheable) {
>> +            time_t  valid;
>> +
>> +            if (rc == NGX_HTTP_BAD_GATEWAY || rc ==
>> NGX_HTTP_GATEWAY_TIME_OUT) {
>> +
>> +                valid = ngx_http_file_cache_valid(u->conf->cache_valid,
>> rc);
>> +
>> +                if (valid) {
>> +                    r->cache->valid_sec = ngx_time() + valid;
>> +                    r->cache->error = rc;
>> +                }
>>             }
>>         }
>>
>
> this a broken patch.  I see another code at
> http://mdounin.ru/hg/nginx-vendor-current/file/1c167244d2fd/src/http/ngx_http_upstream.c#l2991
>

nope, it's my mind is broken. Sorry.

-- 
wbr, Kirill
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://nginx.org/pipermail/nginx-devel/attachments/20110627/92c63982/attachment.html>


More information about the nginx-devel mailing list