avoid calling getThroughSeqnos and getHighSeqnoAndVBUuidFromTarget more than once

Description

In the fix for MB-21812, we call getHighSeqnoAndVBUuidFromTarget() and getThroughSeqnos() instead of commitForCheckpoint() to avoid disk commit on target.  these methods are called only once when we do periodical checkpointing, which is expected. However, they can be called more than once when we do one time checkpointing before pausing replication. This needs to be fixed.

Components

Affects versions

Fix versions

Labels

Environment

None

Link to Log File, atop/blg, CBCollectInfo, Core dump

None

Release Notes Description

None

Activity

Yu Sui June 12, 2017 at 6:16 PM

there are no logs for this and QE verification is not possible/needed.

Arunkumar Senthilnathan June 12, 2017 at 9:27 AM

does this need QE verification? if so, please provide steps

CB robot March 1, 2017 at 1:21 AM

Build 5.0.0-2147 contains goxdcr commit b5b3bbf95605931ed8043dd1b823dcc0ff2ced37 with commit message:
https://couchbasecloud.atlassian.net/browse/MB-23018#icft=MB-23018 avoid calling getHighSeqnoAndVBUuidFromTarget more than once
https://github.com/couchbase/goxdcr/commit/b5b3bbf95605931ed8043dd1b823dcc0ff2ced37

Fixed
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Is this a Regression?

Unknown

Triage

Untriaged

Priority

Instabug

Open Instabug

PagerDuty

Sentry

Zendesk Support

Created February 24, 2017 at 10:27 PM
Updated June 13, 2017 at 5:57 AM
Resolved March 1, 2017 at 12:50 AM
Instabug