SERVER-34051 Require autocommit=false on all transaction operations and add support

for 'startTransaction' argument

This patch requires all operations that are part of a multi-statement transaction to
specify an autocommit=false command argument. It also adds the 'startTransaction'
command argument, which when specified as 'true' indicates that a command is
the beginning of a multi-statement transaction.

1 files changed, 4 insertions, 2 deletions

diff --git a/src/mongo/db/session_catalog.cpp b/src/mongo/db/session_catalog.cpp
index 48869e4c338..d70b68da890 100644
--- a/src/mongo/db/session_catalog.cpp
+++ b/src/mongo/db/session_catalog.cpp
@@ -237,7 +237,8 @@ void SessionCatalog::_releaseSession(const LogicalSessionId& lsid) {
 
 OperationContextSession::OperationContextSession(OperationContext* opCtx,
                                                  bool checkOutSession,
-                                                 boost::optional<bool> autocommit)
+                                                 boost::optional<bool> autocommit,
+                                                 boost::optional<bool> startTransaction)
     : _opCtx(opCtx) {
 
     if (!opCtx->getLogicalSessionId()) {
@@ -265,7 +266,8 @@ OperationContextSession::OperationContextSession(OperationContext* opCtx,
     checkedOutSession->get()->refreshFromStorageIfNeeded(opCtx);
 
     if (opCtx->getTxnNumber()) {
-        checkedOutSession->get()->beginOrContinueTxn(opCtx, *opCtx->getTxnNumber(), autocommit);
+        checkedOutSession->get()->beginOrContinueTxn(
+            opCtx, *opCtx->getTxnNumber(), autocommit, startTransaction);
     }
 }