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

Conversation

@jmd-mongo
Copy link
Collaborator

@jmd-mongo jmd-mongo commented Dec 5, 2024

DESCRIPTION

Documents mongosync latency performance considerations.

JIRA

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

STAGING

Performance -- Network Latency

@netlify
Copy link

netlify bot commented Dec 5, 2024

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

Name Link
🔨 Latest commit c08bcc7
🔍 Latest deploy log https://app.netlify.com/sites/docs-cluster-to-cluster-sync/deploys/6758b9b6139977000852b2b6
😎 Deploy Preview https://deploy-preview-514--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 Author

@jmd-mongo jmd-mongo left a comment

Choose a reason for hiding this comment

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

Thank you for your review, @rohshar! Could you give this one a second look when you have a chance?

Thanks,
Joe

@jmd-mongo jmd-mongo requested a review from rohshar December 10, 2024 17:10
Comment on lines 384 to 385
Mongosync writes operations and updates its own metadata on the
destination cluster in batches in a transaction. This can result in
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
Mongosync writes operations and updates its own metadata on the
destination cluster in batches in a transaction. This can result in
Mongosync does operations and updates its own metadata in batches in a transaction on the
destination cluster. This can result in

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated. Thanks!

cross-shard transactions, which may be more costly if the shards are
far apart.

Latency between the nodes of each individual replica set on the source or destination
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
Latency between the nodes of each individual replica set on the source or destination
Latency between the nodes of any replica set (including shard backing replica set) on the source or destination cluster

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated, but added the parenthetical info to the body of the text to keep the title length manageable.

Copy link
Collaborator Author

@jmd-mongo jmd-mongo 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 your review, @rohshar! This one is ready for another look.

Thanks,
Joe

Comment on lines 384 to 385
Mongosync writes operations and updates its own metadata on the
destination cluster in batches in a transaction. This can result in
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated. Thanks!

cross-shard transactions, which may be more costly if the shards are
far apart.

Latency between the nodes of each individual replica set on the source or destination
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated, but added the parenthetical info to the body of the text to keep the title length manageable.

@jmd-mongo jmd-mongo requested a review from rohshar December 10, 2024 21:31
Copy link
Collaborator

@sarah-olson-mongodb sarah-olson-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 @jmd-mongo! LGTM

@jmd-mongo jmd-mongo merged commit d5e993f into mongodb:master Dec 12, 2024
4 checks passed
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