-
Notifications
You must be signed in to change notification settings - Fork 39
DOCSP-29181 mongosync documentation update #289
DOCSP-29181 mongosync documentation update #289
Conversation
88f3b87 to
94e61f9
Compare
94e61f9 to
21a29d8
Compare
mmaville-mdb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for addressing this one. One small editorial suggestion.
source/reference/mongosync.txt
Outdated
| 6.0 or later. | ||
|
|
||
| You can migrate data on clusters (source) with versions of MongoDB | ||
| lower than 6.0 one time to an Atlas cluster. But this requires |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd re-write the sentence to avoid starting with "But...", perhaps something like "Migration from clusters with a lower version requires additional preperation..." or something along those lines.
mmaville-mdb
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
edobranov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one suggestion on clarity!
source/reference/mongosync.txt
Outdated
| You can migrate data on clusters (source) with versions of MongoDB | ||
| lower than 6.0 one time to an Atlas cluster. Migration from |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The first sentence reads a bit awkwardly to me and I don't think mentioning "one time" is necessary (a migration is inherently a one-time process). I'd consider rewording the sentence to something like:
You can migrate data from clusters with versions of MongoDB lower than 6.0 to an Atlas cluster.
edobranov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM % one last nit, thanks!
Co-authored-by: Evgeni Dobranov <evobranov@gmail.com>
Uh oh!
There was an error while loading. Please reload this page.