MB-17889: No notify from streamRequest 52/60352/3 v3.1.4
authorJim Walker <jim@couchbase.com>
Mon, 22 Feb 2016 21:01:09 +0000 (21:01 +0000)
committerChiyoung Seo <chiyoung@couchbase.com>
Tue, 23 Feb 2016 18:11:09 +0000 (18:11 +0000)
commit4171e5ee93a1febf4cffb90761c350a3ae9d6802
tree120decef2dbc93ddde69451792a0e2449517a4a5
parent674c85e22d03ac9ff31848c3fa5c32dce6859d85
MB-17889: No notify from streamRequest

The 3.1.3 (i.e. before the DCP churn) didn't ever notify from
streamRequest, so that is reverted and helps to bring
view_query latency down (view engine is constantly creating
streams).

A second tweak is to not call stream->next whilst holding
the streamMutex. This can block streamRequest again
affecting the view-engine's DCP stream requests.

Change-Id: I5b57fd7998003251fb32897f37c8a2f15f687a13
Reviewed-on: http://review.couchbase.org/60352
Reviewed-by: Manu Dhundi <manu@couchbase.com>
Tested-by: buildbot <build@couchbase.com>
Well-Formed: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>
Reviewed-by: Chiyoung Seo <chiyoung@couchbase.com>
src/dcp-producer.cc