Welcome! Log In Create A New Profile

Advanced

Re: [PATCH 0 of 5] cache patches

Maxim Dounin
December 12, 2011 05:36AM
Hello!

On Mon, Dec 12, 2011 at 10:25:34AM +0000, António P. P. Almeida wrote:

[...]

> > In the above scenario, request (2) will be served at t=700ms
> > (500ms after request) and request (3) will be served at t=720ms.
> > This is because right now requests use periodic timer (each 500ms)
> > to see if response arrived.
>
> Ok. This is harcoded right now, here I think:
>
> + if (wait) {
> + ngx_add_timer(ev, (timer > 500) ? 500 : timer);
> + return;
> + }
>
> Can we not make this a user defined setting? E.g.:
>
> proxy_cache_lock_request_period <value ms>; # defaults to 500
>
> I don't know if making this period user defined will have consequences
> down the line in other cache manager code.

This may be changed to user setting, but I don't want to as this
is expected to move away from timer to event-based notification
some day.

Maxim Dounin

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH 0 of 5] cache patches

Maxim Dounin 1428 December 11, 2011 10:06AM

[PATCH 1 of 5] Cache: obsolete code removed

Maxim Dounin 511 December 11, 2011 10:06AM

[PATCH 2 of 5] Cache: handling of cache files with long headers

Maxim Dounin 667 December 11, 2011 10:06AM

[PATCH 3 of 5] Cache: only complain on long locked entries

Maxim Dounin 687 December 11, 2011 10:06AM

[PATCH 4 of 5] Cache locks initial implementation

Maxim Dounin 638 December 11, 2011 10:06AM

[PATCH 5 of 5] Cache lock support for fastcgi, scgi, uwsgi

Maxim Dounin 508 December 11, 2011 10:08AM

Re: [PATCH 0 of 5] cache patches

fanboy 625 December 11, 2011 07:00PM

Re: [PATCH 0 of 5] cache patches

Maxim Dounin 484 December 12, 2011 04:54AM

Re: [PATCH 0 of 5] cache patches

António P. P. Almeida 534 December 11, 2011 07:10PM

Re: [PATCH 0 of 5] cache patches

Maxim Dounin 509 December 12, 2011 05:02AM

Re: [PATCH 0 of 5] cache patches

António P. P. Almeida 497 December 12, 2011 05:28AM

Re: [PATCH 0 of 5] cache patches

Maxim Dounin 529 December 12, 2011 05:36AM

Re: [PATCH 0 of 5] cache patches

António P. P. Almeida 817 December 13, 2011 02:42PM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 162
Record Number of Users: 6 on February 13, 2018
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready