MB-19897: Only update sendTime if successfully send noop 78/64878/4
authorDaniel Owen <owend@couchbase.com>
Thu, 21 Apr 2016 09:51:48 +0000 (10:51 +0100)
committerDave Rigby <daver@couchbase.com>
Mon, 13 Jun 2016 10:09:50 +0000 (10:09 +0000)
commitb4007da6ceca5b2bb0902609d6e9c36f1f32e2a3
tree1bc051b39e0b46be49d5450651cc0cdd4102f30f
parent061e360a8f3c2b437e3db3e67556144257495ca8
MB-19897: Only update sendTime if successfully send noop

In the maybeSendNoop function when a DCP producer attempts
to send a noop to a consumer it can receive back
ENGINE_SUCCESS or ENGINE_E2BIG.

We should only set pendingRecv to true and update the
last sendTime if ENGINE_SUCCESS is returned.

Change-Id: Ice8a66dcae35505d7bab7d261f080d5ffb95c8e3
Reviewed-on: http://review.couchbase.org/64878
Well-Formed: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: buildbot <build@couchbase.com>
src/dcp-producer.cc