On May 19, 2014, at 8:37 PM, Brian Geffon <bri...@apache.org> wrote: > In response to this default value, most spdy servers I've seen use 256 for > this value. Take a look at net internals in chrome.
Even 256 seems pretty aggressive. That's letting a single socket queue 256 parallel requests :) > > Brian > > On Monday, May 19, 2014, Leif Hedstrom <zw...@apache.org> wrote: > >> Agree with James, let's be consistent. >> >> -- Leif >> >>> On May 19, 2014, at 8:45 PM, James Peach <jpe...@apache.org> wrote: >>> >>>> On May 19, 2014, at 5:26 PM, a...@apache.org wrote: >>>> >>>> Repository: trafficserver >>>> Updated Branches: >>>> refs/heads/master c25fb7541 -> ce8304309 >>>> >>>> >>>> TS-2821 Add to default records.config and tweak the name to be more >> consistent. >>>> >>>> >>>> Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo >>>> Commit: >> http://git-wip-us.apache.org/repos/asf/trafficserver/commit/ce830430 >>>> Tree: >> http://git-wip-us.apache.org/repos/asf/trafficserver/tree/ce830430 >>>> Diff: >> http://git-wip-us.apache.org/repos/asf/trafficserver/diff/ce830430 >>>> >>>> Branch: refs/heads/master >>>> Commit: ce8304309e2c17b4b3efbe93f990f7106a28a7e1 >>>> Parents: c25fb75 >>>> Author: Alan M. Carroll <a...@network-geographics.com> >>>> Authored: Mon May 19 17:25:29 2014 -0700 >>>> Committer: Alan M. Carroll <a...@network-geographics.com> >>>> Committed: Mon May 19 17:25:29 2014 -0700 >>>> >>>> ---------------------------------------------------------------------- >>>> doc/reference/configuration/records.config.en.rst | 4 ++-- >>>> mgmt/RecordsConfig.cc | 2 +- >>>> proxy/config/records.config.default.in | 6 ++++++ >>>> proxy/spdy/SpdyCommon.cc | 2 +- >>>> 4 files changed, 10 insertions(+), 4 deletions(-) >>>> ---------------------------------------------------------------------- >>>> >>>> >>>> >> http://git-wip-us.apache.org/repos/asf/trafficserver/blob/ce830430/doc/reference/configuration/records.config.en.rst >>>> ---------------------------------------------------------------------- >>>> diff --git a/doc/reference/configuration/records.config.en.rst >> b/doc/reference/configuration/records.config.en.rst >>>> index 4ca5613..cef38fe 100644 >>>> --- a/doc/reference/configuration/records.config.en.rst >>>> +++ b/doc/reference/configuration/records.config.en.rst >>>> @@ -2227,10 +2227,10 @@ ICP Configuration >>>> SPDY Configuration >>>> ================== >>>> >>>> -.. ts:cv:: CONFIG proxy.config.spdy.max_concurrent_streams INT 1000 >>>> +.. ts:cv:: CONFIG proxy.config.spdy.client.max_concurrent_streams INT >> 1000 >>>> :reloadable: >>> >>> This should be proxy.config.spdy.SERVER.max_concurrent_streams. The >> terms "client" and "server" are well established. "proxy.config.ssl.client" >> refers to ATS making outbound SSL connections, and >> "proxy.config.ssl.server" and "proxy.config.http.server" refer to ATS >> accepting sessions from clients. This setting is all about ATS acting as a >> SPDY server. >>> >>> The default value of 1000 is *huge*. 10 would be better IMHO. >>> >>> We should not be adding more default entries to "records.config"; the >> goal is to reduce the number of entries that are duplicated in that file. >>> >>> Finally, I think that it is worth documenting that which this is >> reloadable, only new SPDY sessions will get the new value. Since SPDY >> sessions are long-lasting, this may surprise people. >>> >>>> >>>> - Set the maximum number of concurrent streams per client SPDY >> connection. >>>> + Set the maximum number of concurrent streams per client connection. >>>> >>>> Scheduled Update Co
