Set a better default for maxReadsPerEvent
Summary:
0 (infinity) seems like a bad default for this. Just found a 2~5% latency win in thrift by setting this to something more reasonable. Any reason not to set a better default globally?
Since this is already on for mcrouter/proxygen/thrift, the biggest change would probably be service router
tests postponed longer than 20 minutes
Test Plan:
thrift perf test, multifeed canary
Canary URL: https://our.intern.facebook.com/intern/feedtools/mycanary?id=davejwatson_20141208-102635_local.1418063194_d201eceec04e5e097a6dffb645a424d502db8742_
Reviewed By: afrind@fb.com
Subscribers: doug, net-systems@, njormrod, folly-diffs@, jsedgwick, haijunz, andrewcox, alandau, fugalh
FB internal diff:
D1725142
Signature: t1:
1725142:
1418066284:
c2e1cbf27b0c7a98cf2395f369683bbcce4cdce4
projects / folly.git / commit