diff options
| author | ADAM David Alan Martin <adam.martin@10gen.com> | 2018-10-05 17:59:07 -0400 |
|---|---|---|
| committer | ADAM David Alan Martin <adam.martin@10gen.com> | 2018-10-05 17:59:07 -0400 |
| commit | a55b0131e76846491a23da09f812acede0f7cf46 (patch) | |
| tree | 58521e21d8b3d472e997d2f23e8e8e628fb674a2 /src/mongo/db/s/move_primary_source_manager.cpp | |
| parent | 2e313aa04c07080e0734e694f43f99f4bc8ddb94 (diff) | |
| download | mongo-a55b0131e76846491a23da09f812acede0f7cf46.tar.gz | |
SERVER-37431 Make `logChange` not return `Status`.
This eliminates the need for a lot of superfluous `.ignore()`
calls. It also cleans up a lot of `.transitional_ignore()`
calls.
Diffstat (limited to 'src/mongo/db/s/move_primary_source_manager.cpp')
| -rw-r--r-- | src/mongo/db/s/move_primary_source_manager.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/mongo/db/s/move_primary_source_manager.cpp b/src/mongo/db/s/move_primary_source_manager.cpp index 5b8902455de..89e7ae487b7 100644 --- a/src/mongo/db/s/move_primary_source_manager.cpp +++ b/src/mongo/db/s/move_primary_source_manager.cpp @@ -75,7 +75,7 @@ Status MovePrimarySourceManager::clone(OperationContext* opCtx) { log() << "Moving " << _dbname << " primary from: " << _fromShard << " to: " << _toShard; // Record start in changelog - uassertStatusOK(Grid::get(opCtx)->catalogClient()->logChange( + uassertStatusOK(Grid::get(opCtx)->catalogClient()->logChangeChecked( opCtx, "movePrimary.start", _dbname.toString(), @@ -223,7 +223,7 @@ Status MovePrimarySourceManager::commitOnConfig(OperationContext* opCtx) { "against the config server to obtain its latest optime" << causedBy(redact(commitStatus)); - Status validateStatus = Grid::get(opCtx)->catalogClient()->logChange( + Status validateStatus = Grid::get(opCtx)->catalogClient()->logChangeChecked( opCtx, "movePrimary.validating", getNss().ns(), @@ -285,7 +285,7 @@ Status MovePrimarySourceManager::commitOnConfig(OperationContext* opCtx) { _cleanup(opCtx); - uassertStatusOK(Grid::get(opCtx)->catalogClient()->logChange( + uassertStatusOK(Grid::get(opCtx)->catalogClient()->logChangeChecked( opCtx, "movePrimary.commit", _dbname.toString(), @@ -325,7 +325,7 @@ void MovePrimarySourceManager::cleanupOnError(OperationContext* opCtx) { } try { - uassertStatusOK(Grid::get(opCtx)->catalogClient()->logChange( + uassertStatusOK(Grid::get(opCtx)->catalogClient()->logChangeChecked( opCtx, "movePrimary.error", _dbname.toString(), |