MB-19732: Only update sendTime if successfully send noop 24/64724/8
authorDaniel Owen <owend@couchbase.com>
Thu, 21 Apr 2016 09:51:48 +0000 (10:51 +0100)
committerDave Rigby <daver@couchbase.com>
Tue, 7 Jun 2016 15:14:31 +0000 (15:14 +0000)
commit62563bc5f0c59b823a9ee91c02a9012879fcf083
tree764f92f5acbf57c6c5b069c87a04b2d9e3863ce9
parentd91e1188addb8e52dd02df38c3b4212fb002156a
MB-19732: 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/64724
Well-Formed: buildbot <build@couchbase.com>
Reviewed-by: Manu Dhundi <manu@couchbase.com>
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>
src/dcp-producer.cc