diff options
author | James Wahlin <james.wahlin@10gen.com> | 2016-05-18 09:54:15 -0400 |
---|---|---|
committer | James Wahlin <james.wahlin@10gen.com> | 2016-05-19 14:49:25 -0400 |
commit | 47b0f99814f2c78edd0ae9aebf76f2c4bfddcad6 (patch) | |
tree | 42210bdeec16f0839905d913eb59ed70bc9d58f6 /src/mongo/db/ops/update.cpp | |
parent | 5fb43bea8df4d62c97ad3c4777f773a7c3e032a3 (diff) | |
download | mongo-47b0f99814f2c78edd0ae9aebf76f2c4bfddcad6.tar.gz |
SERVER-23933 Remove OpDebug from ops/update.cpp::update()
Diffstat (limited to 'src/mongo/db/ops/update.cpp')
-rw-r--r-- | src/mongo/db/ops/update.cpp | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/src/mongo/db/ops/update.cpp b/src/mongo/db/ops/update.cpp index 82a816e5b3c..d5b83ae666b 100644 --- a/src/mongo/db/ops/update.cpp +++ b/src/mongo/db/ops/update.cpp @@ -57,10 +57,7 @@ namespace mongo { -UpdateResult update(OperationContext* txn, - Database* db, - const UpdateRequest& request, - OpDebug* opDebug) { +UpdateResult update(OperationContext* txn, Database* db, const UpdateRequest& request) { invariant(db); // Explain should never use this helper. @@ -113,8 +110,9 @@ UpdateResult update(OperationContext* txn, ParsedUpdate parsedUpdate(txn, &request); uassertStatusOK(parsedUpdate.parseRequest()); + OpDebug* const nullOpDebug = nullptr; std::unique_ptr<PlanExecutor> exec = - uassertStatusOK(getExecutorUpdate(txn, opDebug, collection, &parsedUpdate)); + uassertStatusOK(getExecutorUpdate(txn, nullOpDebug, collection, &parsedUpdate)); uassertStatusOK(exec->executePlan()); if (repl::ReplClientInfo::forClient(client).getLastOp() != lastOpAtOperationStart) { @@ -124,11 +122,7 @@ UpdateResult update(OperationContext* txn, lastOpSetterGuard.Dismiss(); } - PlanSummaryStats summaryStats; - Explain::getSummaryStats(*exec, &summaryStats); const UpdateStats* updateStats = UpdateStage::getUpdateStats(exec.get()); - UpdateStage::recordUpdateStatsInOpDebug(updateStats, opDebug); - opDebug->setPlanSummaryMetrics(summaryStats); return UpdateStage::makeUpdateResult(updateStats); } |