SERVER-22657 Make MigrationChunkClonerSourceLegacy not do host resolution

No functional changes, just a cleanup to move the resolution of the
recipient shard's primary out of MigrationChunkClonerSourceLegacy in order
to make it simpler for unit-testing.

1 files changed, 6 insertions, 3 deletions

diff --git a/src/mongo/db/s/migration_chunk_cloner_source_legacy.h b/src/mongo/db/s/migration_chunk_cloner_source_legacy.h
index 8858b3839cb..df14e57b154 100644
--- a/src/mongo/db/s/migration_chunk_cloner_source_legacy.h
+++ b/src/mongo/db/s/migration_chunk_cloner_source_legacy.h
@@ -55,7 +55,10 @@ class MigrationChunkClonerSourceLegacy final : public MigrationChunkClonerSource
     MONGO_DISALLOW_COPYING(MigrationChunkClonerSourceLegacy);
 
 public:
-    MigrationChunkClonerSourceLegacy(MoveChunkRequest request, const BSONObj& shardKeyPattern);
+    MigrationChunkClonerSourceLegacy(MoveChunkRequest request,
+                                     const BSONObj& shardKeyPattern,
+                                     ConnectionString donorConnStr,
+                                     HostAndPort recipientHost);
     ~MigrationChunkClonerSourceLegacy();
 
     Status startClone(OperationContext* txn) override;
@@ -168,10 +171,10 @@ private:
     const MigrationSessionId _sessionId;
 
     // The resolved connection string of the donor shard
-    ConnectionString _donorCS;
+    const ConnectionString _donorConnStr;
 
     // The resolved primary of the recipient shard
-    HostAndPort _recipientHost;
+    const HostAndPort _recipientHost;
 
     // Registered deletion notifications plan executor, which will listen for document deletions
     // during the cloning stage