MB-22178: Don't use opencheckpointid to determine if in backfill phase 13/73813/3
authorDaniel Owen <owend@couchbase.com>
Thu, 8 Dec 2016 15:59:44 +0000 (15:59 +0000)
committerJim Walker <jim@couchbase.com>
Thu, 23 Feb 2017 19:15:57 +0000 (19:15 +0000)
commit17d8153f04d64c5ecb92e1f2e68bbfa7445e521f
treefd3e77693cd5f544cb46196d9460b6938d5077fc
parentd7e179160929f7649f5c5774620ebbec2af94126
MB-22178: Don't use opencheckpointid to determine if in backfill phase

The opencheckpointid of a bucket can be zero after a rollback.
If an opencheckpointid was zero it was assumed that the vbucket was in
backfilling state.  This caused the producer stream request to be stuck
waiting for backfilling to complete.

Ths patch uses the vb->isBackfillPhase() to determine if the vbucket is
in a backfill state as opposed to using an opencheckpointid of zero.

Change-Id: Ia977d6bf90e54fd1ceb8db4a9088b19d94d4bc8c
Reviewed-on: http://review.couchbase.org/70810
Reviewed-by: Manu Dhundi <manu@couchbase.com>
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Jim Walker <jim@couchbase.com>
Reviewed-on: http://review.couchbase.org/73813
Well-Formed: Build Bot <build@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>
src/dcp/producer.cc
src/dcp/stream.cc
src/ep.cc
tests/module_tests/dcp_test.cc
tests/module_tests/evp_store_rollback_test.cc