One-two clarifications
On 11/22/2012 02:10 PM, Amos Jeffries wrote:
> On 22/11/2012 9:14 a.m., Tsantilas Christos wrote:
> * please enable concurrency by default. Since this is a new interface we
> have no legacy excuses to hold us back on good performance.
Not a real concurrency. It is just a testing helper, can not be used in
production.
Is it OK?
>
>
> src/cf.data.pre:
> * missing documentation about concurrency= support on this interface.
> * if the bundled helper is upgraded to concurrency support the config
> default can be changed to enable some number of channels.
Is it OK to set "concurrency=1"
>
>
> src/forward.cc:
> * Please check for BH status being returned by the helper differently
> from the "NULL reply" error case. The handler can receive a reply with
> BH having content. Ssl::CertValidationHelper::sslSubmit() being one
> place which uses a message for Squid-side errors in the helper transaction.
If there is not any problem I will fix this one to "helper validator
cache" patch, to minimize the the conflicts during merging the patches
to trunk.
Received on Thu Nov 22 2012 - 16:57:16 MST
This archive was generated by hypermail 2.2.0 : Fri Nov 23 2012 - 12:00:08 MST