SERVER-36224 Refactor Collection::updateDocument args

Previously, Collection::updateDocument expected to get a namespace and
UUID from its OplogEntryUpdateArgs, but that is not necessary, because
the collection already knows its own namespace and UUID. This caused a
problem in DurableViewCatalogImpl::upsert(), which did not set the
UUID value in the args, leading to oplog entries without their UUID
value.

1 files changed, 1 insertions, 2 deletions

diff --git a/src/mongo/db/views/durable_view_catalog.cpp b/src/mongo/db/views/durable_view_catalog.cpp
index c02b091e88c..3284b303d40 100644
--- a/src/mongo/db/views/durable_view_catalog.cpp
+++ b/src/mongo/db/views/durable_view_catalog.cpp
@@ -149,8 +149,7 @@ void DurableViewCatalogImpl::upsert(OperationContext* opCtx,
         uassertStatusOK(
             systemViews->insertDocument(opCtx, InsertStatement(view), &CurOp::get(opCtx)->debug()));
     } else {
-        OplogUpdateEntryArgs args;
-        args.nss = systemViewsNs;
+        CollectionUpdateArgs args;
         args.update = view;
         args.criteria = BSON("_id" << name.ns());
         args.fromMigrate = false;