SERVER-27329 cleaned up BackgroundSync::_rollback() argument.

Moved dependencies on concrete classes and StorageInterace::get() out of _rollback()
into BackgroundSync::_produce().

This commit also removes the dependency on StorageInterface::get() from syncRollback().

1 files changed, 4 insertions, 1 deletions

diff --git a/src/mongo/db/repl/rs_rollback.h b/src/mongo/db/repl/rs_rollback.h
index 58dd4a27d4f..c5589003ec4 100644
--- a/src/mongo/db/repl/rs_rollback.h
+++ b/src/mongo/db/repl/rs_rollback.h
@@ -44,6 +44,7 @@ namespace repl {
 class OplogInterface;
 class ReplicationCoordinator;
 class RollbackSource;
+class StorageInterface;
 
 /**
  * Initiates the rollback process.
@@ -65,6 +66,7 @@ class RollbackSource;
  *            provides oplog; and
  *            supports fetching documents and copying collections.
  * @param replCoord Used to track the rollback ID and to change the follower state
+ * @param storageInterface Used to update minValid.
  *
  * If requiredRBID is supplied, we error if the upstream node has a different RBID (ie it rolled
  * back) after fetching any information from it.
@@ -77,7 +79,8 @@ Status syncRollback(OperationContext* opCtx,
                     const OplogInterface& localOplog,
                     const RollbackSource& rollbackSource,
                     boost::optional<int> requiredRBID,
-                    ReplicationCoordinator* replCoord);
+                    ReplicationCoordinator* replCoord,
+                    StorageInterface* storageInterface);
 
 /**
  * This namespace contains internal details of the rollback system. It is only exposed in a header