Re: [PATCH] reply_from_cache and reply_to_cache

From: Amos Jeffries <squid3_at_treenet.co.nz>
Date: Thu, 19 Dec 2013 00:16:57 +1300

Bumping this discussion.

On 16/10/2013 4:36 a.m., Alex Rousskov wrote:
>
> Going forward, I think we need to decide:
>
> A) Whether altering the existing "cache" directive semantics is
> desirable. If it is a good idea, we can remove or deprecate that option
> and ignore its [end-of-life] existence when deciding how to structure
> the new directives.
>
> B) Whether we should keep decision point 1 (action before hit/miss
> determination and partial loading of the response is made).
>
>
> I am not sure about (A) but I suspect we may want keep the "fast"
> decision point in (B) if for no other reason than performance of simple
> cases.
>

I can agree with (B) and putting off the review of the "cache" directive
for some future date if necessary.

I've had a read through the version of your patch cleaned up and
submitted as fix for bug 3937. It appears to implement what you are
talking about as (B). Are there any additional changes to be made on that?

I am okay with a +1 on it and those updated names.

Amos
Received on Wed Dec 18 2013 - 11:17:06 MST

This archive was generated by hypermail 2.2.0 : Wed Dec 18 2013 - 12:00:12 MST