SERVER-52609 Append corruptRecords to validate results as an array

5 files changed, 16 insertions, 16 deletions

diff --git a/src/mongo/db/catalog/validate_results.cpp b/src/mongo/db/catalog/validate_results.cpp
index 2c3c9a4b820..820ddee062e 100644
--- a/src/mongo/db/catalog/validate_results.cpp
+++ b/src/mongo/db/catalog/validate_results.cpp
@@ -31,28 +31,28 @@
 
 namespace mongo {
 
-void ValidateResults::appendToResultObj(BSONObjBuilder& resultObj, bool debugging) const {
-    resultObj.appendBool("valid", valid);
-    resultObj.appendBool("repaired", repaired);
+void ValidateResults::appendToResultObj(BSONObjBuilder* resultObj, bool debugging) const {
+    resultObj->appendBool("valid", valid);
+    resultObj->appendBool("repaired", repaired);
     if (readTimestamp) {
-        resultObj.append("readTimestamp", readTimestamp.get());
+        resultObj->append("readTimestamp", readTimestamp.get());
     }
-    resultObj.append("warnings", warnings);
-    resultObj.append("errors", errors);
-    resultObj.append("extraIndexEntries", extraIndexEntries);
-    resultObj.append("missingIndexEntries", missingIndexEntries);
+    resultObj->append("warnings", warnings);
+    resultObj->append("errors", errors);
+    resultObj->append("extraIndexEntries", extraIndexEntries);
+    resultObj->append("missingIndexEntries", missingIndexEntries);
 
     // Need to convert RecordId to int64_t to append to BSONObjBuilder
     BSONArrayBuilder builder;
     for (RecordId corruptRecord : corruptRecords) {
         builder.append(corruptRecord.repr());
     }
-    resultObj.append("corruptRecords", builder.done());
+    resultObj->append("corruptRecords", builder.arr());
 
     if (repaired || debugging) {
-        resultObj.appendNumber("numRemovedCorruptRecords", numRemovedCorruptRecords);
-        resultObj.appendNumber("numRemovedExtraIndexEntries", numRemovedExtraIndexEntries);
-        resultObj.appendNumber("numInsertedMissingIndexEntries", numInsertedMissingIndexEntries);
+        resultObj->appendNumber("numRemovedCorruptRecords", numRemovedCorruptRecords);
+        resultObj->appendNumber("numRemovedExtraIndexEntries", numRemovedExtraIndexEntries);
+        resultObj->appendNumber("numInsertedMissingIndexEntries", numInsertedMissingIndexEntries);
     }
 }
 }  // namespace mongo
diff --git a/src/mongo/db/catalog/validate_results.h b/src/mongo/db/catalog/validate_results.h
index 96752f70877..9e0062d8fb3 100644
--- a/src/mongo/db/catalog/validate_results.h
+++ b/src/mongo/db/catalog/validate_results.h
@@ -68,7 +68,7 @@ struct ValidateResults {
 
     // Takes a bool that indicates the context of the caller and a BSONObjBuilder to append with
     // validate results.
-    void appendToResultObj(BSONObjBuilder& resultObj, bool debugging) const;
+    void appendToResultObj(BSONObjBuilder* resultObj, bool debugging) const;
 };
 
 }  // namespace mongo
diff --git a/src/mongo/db/commands/validate.cpp b/src/mongo/db/commands/validate.cpp
index a7080fb618b..c1d787a49bb 100644
--- a/src/mongo/db/commands/validate.cpp
+++ b/src/mongo/db/commands/validate.cpp
@@ -227,7 +227,7 @@ public:
             return CommandHelpers::appendCommandStatusNoThrow(result, status);
         }
 
-        validateResults.appendToResultObj(result, /*debugging=*/false);
+        validateResults.appendToResultObj(&result, /*debugging=*/false);
 
         if (!validateResults.valid) {
             result.append("advice",
diff --git a/src/mongo/db/repair.cpp b/src/mongo/db/repair.cpp
index efc23ea847b..44c79ad28b4 100644
--- a/src/mongo/db/repair.cpp
+++ b/src/mongo/db/repair.cpp
@@ -245,7 +245,7 @@ Status repairCollection(OperationContext* opCtx,
 
     BSONObjBuilder detailedResults;
     const bool debug = false;
-    validateResults.appendToResultObj(detailedResults, debug);
+    validateResults.appendToResultObj(&detailedResults, debug);
 
     LOGV2(21028,
           "Collection validation",
diff --git a/src/mongo/db/storage/storage_debug_util.cpp b/src/mongo/db/storage/storage_debug_util.cpp
index a17de8927f4..3b4fe5bc98a 100644
--- a/src/mongo/db/storage/storage_debug_util.cpp
+++ b/src/mongo/db/storage/storage_debug_util.cpp
@@ -130,7 +130,7 @@ void printCollectionAndIndexTableEntries(OperationContext* opCtx, const Namespac
 void printValidateResults(const ValidateResults& results) {
     BSONObjBuilder resultObj;
 
-    results.appendToResultObj(resultObj, /*debugging=*/true);
+    results.appendToResultObj(&resultObj, /*debugging=*/true);
 
     LOGV2(51812, "Results", "results"_attr = resultObj.done());
 }