Skip to content
This repository was archived by the owner on May 1, 2025. It is now read-only.

Conversation

ltran-mdb2
Copy link
Collaborator

@ltran-mdb2 ltran-mdb2 commented Nov 7, 2024

DESCRIPTION

Adds info on shared cluster behavior and moves sections to the general mongosync behavior page

STAGING

https://deploy-preview-472--docs-cluster-to-cluster-sync.netlify.app/reference/mongosync/mongosync-behavior/#sharded-clusters
https://deploy-preview-472--docs-cluster-to-cluster-sync.netlify.app/reference/api/start/#behavior

JIRA

https://jira.mongodb.org/browse/DOCSP-45117

SELF-REVIEW CHECKLIST

  • Does each file have 3-5 taxonomy facet tags?
    See the taxonomy tagging instructions and this example PR
  • Is this free of any warnings or errors in the RST?
  • Is this free of spelling errors?
  • Is this free of grammatical errors?
  • Is this free of staging / rendering issues?
  • Are all the links working?

EXTERNAL REVIEW REQUIREMENTS

What's expected of an external reviewer?

Copy link

netlify bot commented Nov 7, 2024

Deploy Preview for docs-cluster-to-cluster-sync ready!

Name Link
🔨 Latest commit 405c10c
🔍 Latest deploy log https://app.netlify.com/sites/docs-cluster-to-cluster-sync/deploys/672e5c3050cd7500088b8d52
😎 Deploy Preview https://deploy-preview-472--docs-cluster-to-cluster-sync.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

@jason-price-mongodb jason-price-mongodb left a comment

Choose a reason for hiding this comment

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

@ltran-mdb2 LGTM with suggestions. Cheers.


Running :dbcommand:`movePrimary` on the source or desintation cluster
during migration may result in a fatal error or require you to
restart the migration from scratch. For more information, see
Copy link
Collaborator

Choose a reason for hiding this comment

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

Consider: "from the start"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

Primary Shards
``````````````

When ``mongosync`` syncs to a sharded destination cluster, it assigns a
Copy link
Collaborator

Choose a reason for hiding this comment

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

Consider clarifying what "it assigns" refers to. Cheers.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

Even if documents move between source shards during replication,
{+c2c-product-name+} maintains the :term:`eventual consistency`
guarantee on the destination cluster. For more information, see
:ref:`c2c-sharded-clusters`.
Copy link
Contributor

Choose a reason for hiding this comment

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

I mentioned in the description of DOCSP-45117 that the balancer / chunk migration stuff no longer applies, so L62- L71 isn't relevant anymore. We can instead 'summarize' the two sub-sections after it by replacing L62 - L71 with:

{+c2c-product-name+} supports replication between sharded clusters. Individual shards are replicated in parallel from the source cluster to the destination cluster, however the source cluster's sharding configuration is not preserved.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done!

@ltran-mdb2 ltran-mdb2 requested a review from edobranov November 8, 2024 17:32
Copy link
Contributor

@edobranov edobranov left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@ltran-mdb2 ltran-mdb2 merged commit 0ab11b3 into mongodb:master Nov 8, 2024
4 checks passed
ltran-mdb2 added a commit to ltran-mdb2/docs-cluster-to-cluster-sync that referenced this pull request Nov 8, 2024
* DOCSP-45117-sharded-cluster-behavior

* DOCSP-45117-sharded-cluster-behavior

* DOCSP-45117 fixing label

* DOCSP-45117 copy edits

* DOCSP-45117 tech edit

* DOCSP-45117 fixing underline

(cherry picked from commit 0ab11b3)
ltran-mdb2 added a commit to ltran-mdb2/docs-cluster-to-cluster-sync that referenced this pull request Nov 8, 2024
* DOCSP-45117-sharded-cluster-behavior

* DOCSP-45117-sharded-cluster-behavior

* DOCSP-45117 fixing label

* DOCSP-45117 copy edits

* DOCSP-45117 tech edit

* DOCSP-45117 fixing underline

(cherry picked from commit 0ab11b3)

# Conflicts:
#	source/reference/api/start.txt
@ltran-mdb2
Copy link
Collaborator Author

💚 All backports created successfully

Status Branch Result
v1.7
v1.8

Questions ?

Please refer to the Backport tool documentation

ltran-mdb2 added a commit that referenced this pull request Nov 8, 2024
* DOCSP-45117-sharded-cluster-behavior

* DOCSP-45117-sharded-cluster-behavior

* DOCSP-45117 fixing label

* DOCSP-45117 copy edits

* DOCSP-45117 tech edit

* DOCSP-45117 fixing underline

(cherry picked from commit 0ab11b3)

# Conflicts:
#	source/reference/api/start.txt
ltran-mdb2 added a commit that referenced this pull request Nov 8, 2024
* DOCSP-45117-sharded-cluster-behavior

* DOCSP-45117-sharded-cluster-behavior

* DOCSP-45117 fixing label

* DOCSP-45117 copy edits

* DOCSP-45117 tech edit

* DOCSP-45117 fixing underline

(cherry picked from commit 0ab11b3)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants