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

Conversation

@ianf-mongodb ianf-mongodb self-requested a review February 15, 2024 18:44
Copy link
Contributor

@ianf-mongodb ianf-mongodb left a comment

Choose a reason for hiding this comment

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

Hi @ltran-mdb2 this content looks good, but I wonder if we should add one additional example, demonstrating this concept. WDYT?

Copy link
Contributor

@ianf-mongodb ianf-mongodb left a comment

Choose a reason for hiding this comment

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

Thanks for the context, LGTM!


.. note::

.. include:: /includes/api/facts/multiple-mongosync-endpoints
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 on all of the API endpoint pages it becomes confusing because it doesn't link to a multiple mongosync section that explains how to use it. Could we try to add something like that?

@@ -0,0 +1,3 @@
When you configure multiple ``mongosync``
instances, you must send identical API endpoint commands to each
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
instances, you must send identical API endpoint commands to each
instances to sync between sharded clusters, you must send identical API endpoint commands to each

@ltran-mdb2
Copy link
Collaborator Author

Thanks for your feedback @rohshar ! Could you take another look when you get a chance?

@sarah-olson-mongodb sarah-olson-mongodb merged commit 27265f5 into mongodb:master Feb 21, 2024
ltran-mdb2 added a commit to ltran-mdb2/docs-cluster-to-cluster-sync that referenced this pull request Feb 22, 2024
* DOCSP-32690 multiple mongosync endpoint note

* DOCSP-32690 splitting up notes

* DOCSP-32690 adding links to multiple mongosyncs sections

* DOCSP-32690 tech edits
sarah-olson-mongodb pushed a commit that referenced this pull request Feb 22, 2024
* DOCSP-32690 multiple mongosync endpoint note

* DOCSP-32690 splitting up notes

* DOCSP-32690 adding links to multiple mongosyncs sections

* DOCSP-32690 tech edits
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.

4 participants