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.
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
