DOCSP-34822: commitTxn is idempotent #337
Conversation
source/fundamentals/transactions.txt
Outdated
| information, see :manual:`startTransaction() | ||
| </reference/method/Session.startTransaction/>` in the Server manual. |
There was a problem hiding this comment.
S: I think these sentences would flow a little better when we say "see the guide (or page) in the..."
Applies to all "For more information..." instances
| information, see :manual:`startTransaction() | |
| </reference/method/Session.startTransaction/>` in the Server manual. | |
| information, see the :manual:`startTransaction() | |
| </reference/method/Session.startTransaction/>` guide in the Server manual. |
|
|
||
| .. note:: Retrying a Transaction | ||
|
|
||
| The ``CommitTransaction()`` method is an idempotent function, which |
There was a problem hiding this comment.
CommitTransaction appears idempotent in experimentation, but I can't find server documentation supporting this. Can we assign someone on the server team to review this PR to ensure this is a true statement?
There was a problem hiding this comment.
Sure - I misunderstood the ticket. I can ask a server engineer to review and confirm.
There was a problem hiding this comment.
Per https://mongodb.slack.com/archives/C0V2Z88D8/p1705590222161309, this function is idempotent, but revising the wording to reflect the Server engineer's suggestion!
* DOCSP-34822: commitTxn is idempotent * vale fixes * Server clarification * JS PR fix * language (cherry picked from commit 40d675e)
* DOCSP-34822: commitTxn is idempotent * vale fixes * Server clarification * JS PR fix * language (cherry picked from commit 40d675e)
* DOCSP-34822: commitTxn is idempotent * vale fixes * Server clarification * JS PR fix * language (cherry picked from commit 40d675e)
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-34822
Staging
Self-Review Checklist