cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] pipelining: Add CURLMOPT_PIPELINE_POLICY_FUNCTION

From: Carlo Wood <carlo_at_alinoe.com>
Date: Tue, 25 Nov 2014 16:55:49 +0100

On Tue, 25 Nov 2014 11:18:10 +0100 (CET)
Daniel Stenberg <daniel_at_haxx.se> wrote:

> On Sun, 23 Nov 2014, Carlo Wood wrote:
>
> > This allows the user to tweak the maximum number of connections per
> > host, the maximum number of requests in a pipeline and to override
> > blacklisting or implement whitelisting on a per host basis (since
> > bundles are only per host, not host:port - which imho is a
> > limitation).
>
> I would even consider it a bug. Services on different ports are
> totally unrelated of each other and we shouldn't combine anything
> over them without prior knowledge. So, I would argue that a host in
> that sense should be a host+port pair.
>
> Perhaps a sensible way is to fix that first?

Would you agree with calling snprintf(buf, bufsz, "%s:%d", host.name,
remote_port); every time a bundle needs to be looked up? Or do you want
something that is faster (but will use more memory for an extra struct
member)?

-- 
Carlo Wood <carlo_at_alinoe.com>
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2014-11-25