Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOCSP-45871-parallel-operation-admonition #414

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gmiller-mdb
Copy link
Collaborator

@gmiller-mdb gmiller-mdb commented Jan 10, 2025

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-45871
Staging - https://docs-mongodbcom-staging.corp.mongodb.com/drivers/docsworker-xlarge/NNNNN/

Self-Review Checklist

  • Is this free of any warnings or errors in the RST?
  • Did you run a spell-check?
  • Did you run a grammar-check?
  • Are all the links working?
  • Are the facets and meta keywords accurate?

Copy link

netlify bot commented Jan 10, 2025

Deploy Preview for docs-golang ready!

Name Link
🔨 Latest commit ed60e5d
🔍 Latest deploy log https://app.netlify.com/sites/docs-golang/deploys/67818fd1d0a26e0008a0da17
😎 Deploy Preview https://deploy-preview-414--docs-golang.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@rachel-mack rachel-mack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it should be moved

@@ -195,6 +195,8 @@ For an additional example using sessions and transactions with the {+driver-shor
:website:`developer blog post on Multi-Document ACID Transactions
</developer/languages/go/golang-multi-document-acid-transactions/>`.

.. sharedinclude:: dbx/transactions-parallelism.rst
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this would make more sense to include at the end of the Session and Transaction Options
section, just before the example. People will be looking for this type of information when they're specifying options.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants