SERVER-39705: Always observe multikey changes in IndexBuildInterceptor::sideWrite.

1 files changed, 7 insertions, 4 deletions

diff --git a/src/mongo/db/index/index_build_interceptor.cpp b/src/mongo/db/index/index_build_interceptor.cpp
index 715c4c60fb1..8b8ae089912 100644
--- a/src/mongo/db/index/index_build_interceptor.cpp
+++ b/src/mongo/db/index/index_build_interceptor.cpp
@@ -381,11 +381,10 @@ Status IndexBuildInterceptor::sideWrite(OperationContext* opCtx,
     // `multikeyMetadataKeys` when inserting.
     *numKeysOut = keys.size() + (op == Op::kInsert ? multikeyMetadataKeys.size() : 0);
 
-    if (*numKeysOut == 0) {
-        return Status::OK();
-    }
-
     {
+        // SERVER-39705: It's worth noting that a document may not generate any keys, but be
+        // described as being multikey. This step must be done to maintain parity with `validate`s
+        // expectations.
         stdx::unique_lock<stdx::mutex> lk(_multikeyPathMutex);
         if (_multikeyPaths) {
             MultikeyPathTracker::mergeMultikeyPaths(&_multikeyPaths.get(), multikeyPaths);
@@ -396,6 +395,10 @@ Status IndexBuildInterceptor::sideWrite(OperationContext* opCtx,
         }
     }
 
+    if (*numKeysOut == 0) {
+        return Status::OK();
+    }
+
     std::vector<BSONObj> toInsert;
     for (const auto& key : keys) {
         // Documents inserted into this table must be consumed in insert-order.