Skip to content

DOCS-3443 - clarification re: user for db.copyDatabase() with --auth#1986

Closed
mpaik wants to merge 1 commit into
mongodb:masterfrom
mpaik:DOCS-3443
Closed

DOCS-3443 - clarification re: user for db.copyDatabase() with --auth#1986
mpaik wants to merge 1 commit into
mongodb:masterfrom
mpaik:DOCS-3443

Conversation

@mpaik

@mpaik mpaik commented Sep 15, 2014

Copy link
Copy Markdown
Contributor

No description provided.

tychoish
tychoish reviewed Sep 16, 2014
View reviewed changes
Comment thread source/includes/fact-copydb-behavior.rst

@tychoish tychoish Sep 16, 2014

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

shouldn't "be executed with" -> "specifying"

@mpaik mpaik Sep 16, 2014

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

? "db.copyDatabase() must specifying the credentials of a user present in the source"?

Maybe s/with/using/ or s/with/specifying/? Or is that what you meant to begin with?

@tychoish tychoish Sep 16, 2014

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

sorry, grammatical integration issue.

"you must specify the credentials to |copydb| of a user present in the source database."

"executed with" makes me think of the state at the call site rather than the arguments. to the function.

@mpaik mpaik Sep 17, 2014

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Perhaps s/executed/called/ then? Otherwise your sentence seems fine with the prepositional phrases swapped - "you must specify the credentials of a user present in the source database [when calling] |copydb|." which keeps the 'of a user' more closely modifying 'credentials.' I know you love your second person replacements of passive clauses. =)

@tychoish tychoish closed this Sep 23, 2014
mongo-cr-bot pushed a commit that referenced this pull request Oct 7, 2022
@jmd-mongo
DOCS-15363 documents moveRange (#1947) (#1986)
1f800d9 
* documents moveRange

* review feedback

* restores moveChunk() method

* cleanup

* toShard

* removes moveChunk

* updates

* revert 2.6

* revert

* review feedback

* review feedback
mongodb-server-docs-sync-bot Bot pushed a commit that referenced this pull request Jun 26, 2025
@JuliaMongo
(DOCSP-45404) Remove 1.23 from the compat tables (#1986)
6cfc167
mongodb-server-docs-sync-bot Bot pushed a commit that referenced this pull request Jun 26, 2025
@JuliaMongo @jwilliams-mongo
(DOCSP-45404) Remove 1.23 from the compat tables (#1986)
5d4705d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mpaik @tychoish